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

[Community PR] Integrate merrp's followers branch #4022

Merged

Conversation

AsparagusEduardo
Copy link
Collaborator

@AsparagusEduardo AsparagusEduardo commented Jan 18, 2024

Description

This PR integrates @aarant's followers-expanded-id branch.
It includes Dynamic Overworld Palettes and Expanded Overworld IDs.

Details here: https://github.com/aarant/pokeemerald

Sprites taken from HGSS, with exception of onces that exceed 32x32 dimensions.
New custom 32x32 sprites by LarryTurbo on DeviantArt

TODO list:

  • Add follower sprites to debug menu.
  • Fix 64x64 sprites.
  • Add config to disable following Pokémon and their graphics/object events.
  • Move follower data to gSpeciesInfo and graphics to src/data/pokemon/graphics.
  • Clean up code style to be consistent with the rest of the repository.

Being worked on by @aarant

  • Add Substitute Doll follower sprite as a placeholder for disabled/non-existent followers (commisioned to an artist)
  • Implement asymetric support for followers

Being worked on by Gerben BSc on Discord

  • Fix follower sprite palettes from changes that have happened since last year.

Non-blocking:

  • Decouple following Pokémon from the first slot of the party
    • Add party menu option to choose what Pokémon is following.
    • Config to not have follower if their sprite is not available

Images

mGBA_56sB3dyV05 mGBA_0jUngMUjDZ

Feature(s) this PR does NOT handle:

  • Does not include the lighting-expanded-id or icons branches.
    • lighting-expanded-id will be handled in a separate PR after this one is merged.
  • Link contests should be checked to make sure that they aren't broken by loading other player's palettes dynamically.
  • This PR keeps the follower branch's callfunc as a sorta-duplicate of our current callnative. Main difference is that callfunc can return a value. In a follow up PR, we need to update callnative to also have this and remove callfunc.
    • ScrFunc_ functions should be renamed as well. Done, callfunc functions were converted to callnatives
  • Check integration with @ghoulslash's "follow_me" branch (mostly likely it might need something like "follower Pokémon hides when NPC follower is following").

Discord contact info

AsparagusEduardo

aarant and others added 30 commits December 23, 2021 04:13
Was caused by quick `applymovements` not giving enough time to move the follower on top of the player.
Fixed minor graphical bug in LoadWhiteFlashPalette.
aarant and others added 9 commits May 21, 2024 22:23
Fixes follower pokeball gfx showing garbage for 1 frame when emerging
# Conflicts:
#	src/data/graphics/pokemon.h
#	src/data/pokemon/species_info.h
#	src/data/pokemon/species_info/gen_1_families.h
#	src/data/pokemon/species_info/gen_2_families.h
#	src/data/pokemon/species_info/gen_4_families.h
#	src/data/pokemon/species_info/gen_5_families.h
#	src/data/pokemon/species_info/gen_8_families.h
asm/macros/event.inc Outdated Show resolved Hide resolved
asm/macros/event.inc Outdated Show resolved Hide resolved
data/scripts/field_move_scripts.inc Outdated Show resolved Hide resolved
data/scripts/follower.inc Outdated Show resolved Hide resolved
graphics/weather/fog.pal Outdated Show resolved Hide resolved
src/event_object_movement.c Outdated Show resolved Hide resolved
src/event_object_movement.c Outdated Show resolved Hide resolved
src/event_object_movement.c Outdated Show resolved Hide resolved
src/event_object_movement.c Outdated Show resolved Hide resolved
src/event_object_movement.c Outdated Show resolved Hide resolved
data/tilesets/secondary/rustboro/metatile_attributes.bin Outdated Show resolved Hide resolved
src/event_object_movement.c Outdated Show resolved Hide resolved
src/event_object_movement.c Outdated Show resolved Hide resolved
aarant and others added 6 commits June 1, 2024 14:46
…to _RHH/pr/upcoming/merrpFollowers

# Conflicts:
#	src/event_object_movement.c
#	src/scrcmd.c
This reverts commit 24913f5.

# Conflicts:
#	src/data/pokemon/species_info/gen_4_families.h
#	src/data/pokemon/species_info/gen_5_families.h
#	src/data/pokemon/species_info/gen_8_families.h
@Bassoonian Bassoonian merged commit f11826b into rh-hideout:upcoming Jun 2, 2024
1 check passed
@AsparagusEduardo AsparagusEduardo deleted the _RHH/pr/upcoming/merrpFollowers branch June 2, 2024 15:28
@pkmnsnfrn
Copy link
Collaborator

@ghoulslash asked me to put this here, so I am

List of files changed
Patch Diff

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.

7 participants