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

Configuration to scale Critical Capture odds based on local Pokedex or estimated National Pokedex #6250

Merged

Conversation

surskitty
Copy link

If you are disabling a wide variety of species across generations, it is likely that most pokemon in the National Pokedex will be unobtainable, which means that it's undesirable behavior to scale the % chance that a Critical Capture will happen based off of the number of Pokemon in the national dex assuming all Pokémon were obtainable within that generation. This instead adds a configuration setting to change it to go by the number of Pokemon in the Hoenn Pokedex instead, as it's far easier to adjust what's in the Hoenn Pokedex than to account for disabled species in the National Pokedex.

Discord contact info

wildvenonat

@Bassoonian
Copy link
Collaborator

You'll want to target upcoming instead with this, haha

@surskitty surskitty changed the base branch from master to upcoming February 12, 2025 22:24
@surskitty
Copy link
Author

Oops! That's fixed now :)

@Pawkkie Pawkkie added category: pokemon Pertains to Pokémon data and functionality new-feature Adds a feature labels Feb 12, 2025
@pkmnsnfrn pkmnsnfrn self-assigned this Feb 26, 2025
@pkmnsnfrn
Copy link
Collaborator

@surskitty do you have any artifact showing that this works in game? even a debug print for each of the states showing the odds changing would be great

@surskitty surskitty force-pushed the crit_capture_pokedex_scaling branch from f911595 to a7969c5 Compare February 27, 2025 20:01
@surskitty
Copy link
Author

@surskitty do you have any artifact showing that this works in game? even a debug print for each of the states showing the odds changing would be great

I checked a few of the thresholds that should have given me different odds and they appeared to be functioning correctly, at least. I no longer have record for a few of them because I hadn't taken into account the overall catching odds so the critical capture rates were a flat number, but they were 0 in the correct ballparks for nationaldex up to gen 9 and not-zero in the correct ballparks for hoenndex. Hoenndex stops being 0 at around 10 caught, and gen 9 nationaldex at around 48 (which is what the expansion already does).

[INFO] GBA Debug: Overall catching odds are 243
[INFO] GBA Debug: With 50 caught and totalDexCount 1025, total critical capture odds are 20

[INFO] GBA Debug: Overall catching odds are 244
[INFO] GBA Debug: With 50 caught and totalDexCount 215, total critical capture odds are 40

As far as I'm aware there isn't much known data for canon thresholds based on dex size and the critical capture rates in the expansion are extrapolated to begin with, hehe.

@surskitty surskitty force-pushed the crit_capture_pokedex_scaling branch from a7969c5 to 850609e Compare February 27, 2025 21:27
@pkmnsnfrn pkmnsnfrn merged commit fb28206 into rh-hideout:upcoming Feb 27, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: pokemon Pertains to Pokémon data and functionality new-feature Adds a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants