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 Red and Blue: 0.4.5 Fixes #3106

Merged

Conversation

Alchav
Copy link
Contributor

@Alchav Alchav commented Apr 6, 2024

What is this fixing or adding?

  • After making a fix to the route 4 subregions in the last update, the item location wound up in the wrong one. This fixes that.
  • Fixes a bug in the loop for 10 attempts to get Door Shuffle done, which caused a crash on the third attempt due to the second run corrupting the cache since it wasn't copied.
  • Fixes a bug in the client. I confused the Route 18 Gate script value with the Route 18 script value. Assumption of max valid value was based on Route 18, leading to the client reporting a crash when being stopped by the route 18 guard when you have no bicycle.
  • Fixes an issue with verification of available HM moves. In order to preserve expectations you'd have with various combinations of settings, this verification is now in a function verify_hm_moves which is called both during pre_fill, and after badge placement in fill_hook. fill_hook gets two attempts to place badges without verifying hm moves, after which it starts collecting all pokemon into the state. This will happen if the Pokémons' locations and learnsets are not allowing badges to place, and hm move verification will fix it afterwards. TL;DR this should fix badges failing to place when badgesanity is off + door shuffle is off or on simple.

How was this tested?

The Door Shuffle loop fix and hm verification fix were tested with yamls and seeds that were failing on main. The other fixes have not been tested yet but should be right.

@github-actions github-actions bot added the waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. label Apr 6, 2024
@alwaysintreble alwaysintreble added the is: bug/fix Issues that are reporting bugs or pull requests that are fixing bugs. label Apr 6, 2024
@Alchav Alchav changed the title Pokémon Red and Blue: Move Route 4 item to correct subregion Pokémon Red and Blue: Move Route 4 item to correct subregion + client fix Apr 6, 2024
@Alchav Alchav changed the title Pokémon Red and Blue: Move Route 4 item to correct subregion + client fix Pokémon Red and Blue: Fixes Apr 6, 2024
@Alchav Alchav changed the title Pokémon Red and Blue: Fixes Pokémon Red and Blue: 0.4.5 Fixes Apr 6, 2024
@Berserker66 Berserker66 merged commit 7e660db into ArchipelagoMW:main Apr 13, 2024
15 checks passed
EmilyV99 pushed a commit to EmilyV99/Archipelago that referenced this pull request Apr 15, 2024
EmilyV99 pushed a commit to EmilyV99/Archipelago that referenced this pull request Apr 15, 2024
qwint pushed a commit to qwint/Archipelago that referenced this pull request Jun 24, 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. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants