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

Expansion version in save file #3829

Closed
Ninjdai1 opened this issue Dec 27, 2023 · 2 comments
Closed

Expansion version in save file #3829

Ninjdai1 opened this issue Dec 27, 2023 · 2 comments
Labels
feature-request Requests a new feature

Comments

@Ninjdai1
Copy link

Description

TL;DR: Similar to #3454 in the save file

The goal of this would be to make the expansion's version accessible to programs similar to pkhex:
I'm currently in the making of an expansion-compatible savefile editor. Problem is, some mons and items are not available depending on the version, which would probably cause issues if a mon, moves or items were added in a version they weren't in. Also, if #3438 or similar were to get merged, it would make that kind of software able to change the offsets it uses to get the data.

I know, as it would most likely change a saveblock, this will probabl get rejected, but I still wanted to bring the idea.
Thanks for reading :3

Discord contact info

ninjdai

@Ninjdai1 Ninjdai1 added the feature-request Requests a new feature label Dec 27, 2023
@mrgriffin
Copy link
Collaborator

mrgriffin commented Dec 27, 2023

You probably want your save editor to open both the ROM and the save, and that way you could read the Expansion version out of the RHH ROM header. The GF ROM header will give you pointers to things like gMoveNames and gSpeciesInfo which can help you identify which moves/species/etc are available.

Of course if a hack has added additional stuff to gSpeciesInfo or whatever, then the Expansion version isn't enough to tell you how to interpret the data, and atm there's nothing in the ROM or save which would.


That said, if/when save migration lands there's a chance it could store both the Expansion version and the hack version. The most recent draft PR only stored the hack version though.

@mrgriffin
Copy link
Collaborator

Oh, you might need things like MOVES_COUNT added to the RHH ROM header so that you don't have to resort to hacky things with gMoveNames to work out how many moves there are—feel free to raise a PR with that :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Requests a new feature
Projects
None yet
Development

No branches or pull requests

2 participants