-
Notifications
You must be signed in to change notification settings - Fork 657
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
Pokemon Emerald: Fix possible dexsanity/legendary hunt softlock #3443
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes LGTM and match my understanding of the game. Generated on main
until running into a seed that relied on this logic error, regenerated using the same seed and the error was fixed
Co-authored-by: Exempt-Medic <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes LGTM
…ipelagoMW#3443) * Pokemon Emerald: Remove mirage tower from allowed dexsanity maps * Pokemon Emerald: Prevent placing wailord/relicanth in out of logic maps * Pokemon Emerald: Clarify docstring Co-authored-by: Exempt-Medic <[email protected]> * Pokemon Emerald: Update changelog --------- Co-authored-by: Exempt-Medic <[email protected]>
…ipelagoMW#3443) * Pokemon Emerald: Remove mirage tower from allowed dexsanity maps * Pokemon Emerald: Prevent placing wailord/relicanth in out of logic maps * Pokemon Emerald: Clarify docstring Co-authored-by: Exempt-Medic <[email protected]> * Pokemon Emerald: Update changelog --------- Co-authored-by: Exempt-Medic <[email protected]>
…ipelagoMW#3443) * Pokemon Emerald: Remove mirage tower from allowed dexsanity maps * Pokemon Emerald: Prevent placing wailord/relicanth in out of logic maps * Pokemon Emerald: Clarify docstring Co-authored-by: Exempt-Medic <[email protected]> * Pokemon Emerald: Update changelog --------- Co-authored-by: Exempt-Medic <[email protected]>
…ipelagoMW#3443) * Pokemon Emerald: Remove mirage tower from allowed dexsanity maps * Pokemon Emerald: Prevent placing wailord/relicanth in out of logic maps * Pokemon Emerald: Clarify docstring Co-authored-by: Exempt-Medic <[email protected]> * Pokemon Emerald: Update changelog --------- Co-authored-by: Exempt-Medic <[email protected]>
…ipelagoMW#3443) * Pokemon Emerald: Remove mirage tower from allowed dexsanity maps * Pokemon Emerald: Prevent placing wailord/relicanth in out of logic maps * Pokemon Emerald: Clarify docstring Co-authored-by: Exempt-Medic <[email protected]> * Pokemon Emerald: Update changelog --------- Co-authored-by: Exempt-Medic <[email protected]>
What is this fixing or adding?
Mirage Tower disappears permanently if the player picks up the fossil at the top, so its encounters should not be considered logical access to a species. This counts for dexsanity and for placement of Wailord and Relicanth for legendary hunt.
How was this tested?
Generated a few times with dexsanity and legendary hunt goal.