Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gender differences sprites, part 1 #3070

Merged
merged 6 commits into from
Jun 26, 2023

Conversation

SubzeroEclipse
Copy link

Description

First PR that adds sprites for some Pokémon with gender based differences, namely:

  • Pikachu/Raichu
  • Gloom/Vileplume
  • Doduo/Dodrio
  • Goldeen/Seaking
  • Xatu
  • Politoed
  • Meditite/Medicham
  • Gible/Gabite/Garchomp

Pikachu and Gloom's back sprites were taken from Chaos Rush's resource; Pikachu's front was done by CyanSixFour.

Discord contact info

SubzeroEclipse#2104

Added sprites for:
- Pikachu/Raichu
- Doduo/Dodrio
- Goldeen/Seaking
- Xatu
- Politoed
- Meditite/Medicham
- Gible/Gabite/Garchomp

NOTE: I had to tweak Doduo's palette file because it wasn't matching the normal one and a result Doduo's beak looked weird.
Replaced them with the ones provided by CyanSixFour because they looked better than the ones that I had.
src/data/pokemon/species_info.h Outdated Show resolved Hide resolved
Edited the macros so that only base Pikachu would show different forms based on gender. Big thanks to AsparagusEduardo for helping with this.
@SubzeroEclipse
Copy link
Author

Fixed and ready for review

Copy link
Collaborator

@AsparagusEduardo AsparagusEduardo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good in-game, though #3051 made us realize that bit depths for some sprites were wrong.
You'll need to solve conflicts and fix the bit depths of the following sprites due to them being copied from the erroneous previous depths:

  • Front Female Dodrio
  • Front Female Xatu

Replaced Goldeen (female back), Politoad (female back), Medicham (female all) sprites with those provided by CyanSixFour because they looked better. Also tweaked my male Meditite back
@SubzeroEclipse
Copy link
Author

Ready for re-review! Fixed conflicts and Dodrio/Xatu sprites and swapped sprites as discussed on Discord

@AsparagusEduardo AsparagusEduardo merged commit a44af6f into rh-hideout:upcoming Jun 26, 2023
@SubzeroEclipse SubzeroEclipse deleted the gender-diffs branch July 13, 2023 07:57
@AsparagusEduardo AsparagusEduardo mentioned this pull request Jul 19, 2023
99 tasks
@AsparagusEduardo AsparagusEduardo mentioned this pull request Sep 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants