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

Pokémon R/B: Fix broken options #2162

Merged
merged 2 commits into from
Sep 10, 2023

Conversation

Alchav
Copy link
Contributor

@Alchav Alchav commented Sep 9, 2023

What is this fixing or adding?

  1. Fixes key_items_only which was attempting to set dexsanity to an invalid value due to a last-minute renaming of the choices, and fixes a crash due to an attempt to check gym leader TM locations to set TM moves.
  2. Fixes a missing check for Moon Stone on Key Items checks, which caused setting the Key Items conditions to 100 to lead to generation failure.

How was this tested?

Generating a key_items_only game with 100% key items required

tm = int(item_name[2:4])
move = poke_data.moves[self.local_tms[tm - 1]]["id"]
data[rom_addresses["Gym_Leader_Moves"] + (2 * i)] = move
except KeyError:
Copy link
Member

Choose a reason for hiding this comment

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

I will once again voice my dislike for except *: pass and leave it at that.

@Berserker66 Berserker66 merged commit 0e21a3e into ArchipelagoMW:main Sep 10, 2023
@ThePhar ThePhar added the is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. label Oct 16, 2023
FlySniper pushed a commit to FlySniper/Archipelago that referenced this pull request Nov 14, 2023
Jouramie pushed a commit to Jouramie/Archipelago that referenced this pull request Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants