Skip to content

Fix incorrect encounters in Gen IV games#1601

Merged
Naramsim merged 4 commits into
PokeAPI:masterfrom
Machine-Maker:fix/ef106328f
Jul 13, 2026
Merged

Fix incorrect encounters in Gen IV games#1601
Naramsim merged 4 commits into
PokeAPI:masterfrom
Machine-Maker:fix/ef106328f

Conversation

@Machine-Maker

@Machine-Maker Machine-Maker commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

Change description

The commit ef10632 was not implemented correctly. It incorrectly put a bunch of SoulSilver encounters in the platinum game, leading to encounter confusion (platinum showed a lapras encounter in union cave, a location only in HG/SS for example).

It also added a incorrect "event arceus in party" to the HG lapras encounter. It added it to one-to-many encounters, not just the ones it meant to.

You can see in the original commit, encounters added for games 14 and 15, but 14 is platinum.

AI coding assistance disclosure

Claude helped me find the issue after I noticed the errant Lapras encounter

Contributor check list

  • I have written a description of the contribution and explained its motivation.
  • I have written tests for my code changes (if applicable).
  • I have read and understood the AI Assisted Contribution guidelines.
  • I will own this change in production, and I am prepared to fix any bugs caused by my code change.

@Machine-Maker Machine-Maker changed the title Fix issues with ef106328f Fix incorrect encounters in Gen IV games Jul 11, 2026

@jemarq04 jemarq04 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good catch! Your changes are good, I just noticed an issue with the pre-existing pokemon IDs if you can fix that up. Thanks!

Comment thread data/v2/csv/encounters.csv
@Machine-Maker

Copy link
Copy Markdown
Contributor Author

fixed

@jemarq04 jemarq04 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great, looks good. Please also sync it with the master branch - if you have some difficulties let me know, as I was also experiencing some issues with that using git. I've had to sync with GitHub lately.

@Machine-Maker

Copy link
Copy Markdown
Contributor Author

Done

@Machine-Maker

Copy link
Copy Markdown
Contributor Author

I added another fix. There was no Giratina encounters at all in platinum, when there are 2 (one conditional on not having done the first).

@Naramsim Naramsim merged commit 128bcca into PokeAPI:master Jul 13, 2026
19 checks passed
@Naramsim

Copy link
Copy Markdown
Member

Thanks!

@pokeapi-machine-user

Copy link
Copy Markdown

A PokeAPI/api-data refresh has started. In ~45 minutes the staging branch of PokeAPI/api-data will be pushed with the new generated data.

The staging branch will be deployed in our staging environment and the entire API will be ready to review.

A Pull Request (master<-staging) will be also created at PokeAPI/api-data and assigned to the PokeAPI Core team to be reviewed. If approved and merged new data will soon be available worldwide at pokeapi.co.

@pokeapi-machine-user

Copy link
Copy Markdown

The updater script has finished its job and has now opened a Pull Request towards PokeAPI/api-data with the updated data.

The Pull Request can be seen deployed in our staging environment when CircleCI deploy will be finished (check the start time of the last build).

Naramsim pushed a commit to PokeAPI/api-data that referenced this pull request Jul 13, 2026
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.

4 participants