Skip to content

Add Gen IV Great Marsh changing encounters#1499

Open
Machine-Maker wants to merge 2 commits intoPokeAPI:masterfrom
Machine-Maker:feature/gen-iv-great-marsh
Open

Add Gen IV Great Marsh changing encounters#1499
Machine-Maker wants to merge 2 commits intoPokeAPI:masterfrom
Machine-Maker:feature/gen-iv-great-marsh

Conversation

@Machine-Maker
Copy link
Copy Markdown
Contributor

Change description

Closes #1496

  • Adds new condition type for great marsh selection, x of 32 options.
  • Adds encounters for all the changing pokemon, taking into account pre and post national dex states.

AI coding assistance disclosure

Claude Code helped me analyze decompiled diamond and platinum repos. Also generated the big huge changes based on how they work.

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.

Copy link
Copy Markdown
Member

@jemarq04 jemarq04 left a comment

Choose a reason for hiding this comment

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

I've been taking a look at these but I think before I do an in-depth review I'd like you to manually go through these yourself. From an initial glance, rates seem to be incorrect and I'm already seeing duplicates from my first look at Area 1 (e.g. Wooper).

I've made a couple other comments below.

176,17,friend-safari-slot-1,1
177,17,friend-safari-slot-2,1
178,17,friend-safari-slot-3,1
179,18,great-marsh-daily-slot-0-of-32,1
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.

I'm not sure I like how these are named (but I'm not sure how else to describe the difference in rates) so we can leave that. That being said, what would a rate of 0/32 for a daily spawn mean? I think this was likely a mistake.

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.

Ah, if this is for the default field I'd rather something similar to the story-progress condition and name it great-marsh-daily-slot-none, as if you don't check the binoculars you don't have any daily spawn added in to the encounter pool.

Suggested change
179,18,great-marsh-daily-slot-0-of-32,1
179,18,great-marsh-daily-slot-none,1

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm fine with either really, but I just feel like none doesn't really convey that this is impossible. Like this condition is never true. It's my understanding that there is always a selected pokemon in each area and so the 2 5% encounter slots for the default pokemon are just never ever true. So I don't know if none is the best way to convey that (maybe 0-of-32 isn't the best either).

Comment on lines +302 to +309
179,9,0 of 32
180,9,1 of 32
181,9,2 of 32
182,9,3 of 32
183,9,5 of 32
184,9,Before National Pokédex
185,9,4 of 32
186,9,15 of 32
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.

I think these should be a bit more verbose, like the other entries in the file

@Machine-Maker
Copy link
Copy Markdown
Contributor Author

Machine-Maker commented Apr 25, 2026

It might look like there are duplicates, but that's just cause there are 2 encounter slots for each area. So we need them to be possible on each slot.

Say for diamond, for example. Each of the "special" pokemon, when selected, have a 10% encounter rate in that area. It's the same for every pokemon. But that 10% is distributed across 2 different encounter slots. This is because it decreases the encounter rate for 2 separate pokemon by 5% each (by replacing that slot). So there are a lot of what seem like duplicates, but they are just in different encounter slots.

The same is true for platinum, there are 2 5% "wooper" encounters that are replaced by the 2 5% encounters for the selected pokemon.

For D/P, its either encounter_slot_id 7 or 8

For Plat, its either encounter_slot_id 129 or 130.

@Machine-Maker Machine-Maker force-pushed the feature/gen-iv-great-marsh branch from d4d7323 to ffddda7 Compare April 25, 2026 05:04
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.

Great marsh in D/P/Pt missing encounters for the changing pokemon

2 participants