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

Handle scalars for HOME Keldeo & Meltan; use coded AbilityType for LGPE simulated cards #4446

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Manu098vm
Copy link
Contributor

The AbilityType for LGPE wondercards was hardcoded to 3, to ensure that the ability of generated entities was picked randomly (Any12). From what I’ve seen, the Meltan distributed via the HOME gift always has Ability 1. To make sure the first ability slot is always chosen during generation, the program needs to properly read the AbilityType from the wondercard. I’ve set the wondercard to contain AbilityType 0, so that 'OnlyFirst' is selected.

The handling for height, weight, and scale follows the same approach as the other classes.

Meltan is particularly tricky. The mystery gift stores the Souvenir Ribbon in the core data of the PH3. However, the WB7 and PB7 classes do not currently handle ribbons. It might be possible to utilize unused space within the WB7 and PB7 data, but the code flow would need to be reworked. I’m not sure how we should proceed to properly account for that ribbon.

@Manu098vm
Copy link
Contributor Author

About the Ribbon Issue:
Afaik, ribbons are actually stored in PB7 data when a Pokémon is transferred from HOME to LGPE. See how the Souvenir Ribbon is retained when transferring the Meltan HOME -> LGPE -> HOME -> SwSh.

Hence, the simplest solution may be to implement proper ribbon handling—along with transfer logic to other games—at least for the Souvenir Ribbon, in both the WB7 and PB7 classes.

Other ribbons wouldn't be strictly necessary, as there’s no way to obtain them in a PB7 file.

This approach would ensure that we have a legal PB7 containing the correct ribbon data, and also allows us to check ribbon legality in Gen8+ games, even if the original data comes from a WB7.

I can eventually look into it in the coming days, but it'll probably be later than your planned eta timeline.

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.

1 participant