Skip to content

New encounter tier algorithm#274

Open
Someon1e wants to merge 8 commits intoh0tp-ftw:mainfrom
Someon1e:balanced-luck
Open

New encounter tier algorithm#274
Someon1e wants to merge 8 commits intoh0tp-ftw:mainfrom
Someon1e:balanced-luck

Conversation

@Someon1e
Copy link
Collaborator

Proof-of-concept fix for #273.

@Someon1e Someon1e requested a review from h0tp-ftw January 13, 2026 19:27
@Someon1e Someon1e marked this pull request as ready for review January 17, 2026 13:25
@Someon1e
Copy link
Collaborator Author

Someon1e commented Jan 17, 2026

Looking for feedback on the algorithm - I believe it solves the issue, but there might be other ways to do this.

@marbaret
Copy link

Looking for feedback on the algorithm - I believe it solves the issue, but there might be other ways to do this.

I’ve been using it for two days now, and it solved the issue. The odds feel natural with it

@marbaret
Copy link

marbaret commented Jan 18, 2026

After using it a bit more, I think lowering the odds for Ultra and Baby Pokémon would be nice. I’m currently level 11, and it feels like I keep getting those types of Pokémon way too often.

@h0tp-ftw
Copy link
Owner

committed my prob visualizer if it helps others with visualization, playing with it. the scaling is beautiful and gradual, i suppose some more refinement will take place with user feedback.

level_thresholds = {"Ultra": 30, "Legendary": 50, "Mythical": 75}
for tier in level_thresholds:
# let high level players have high tiers even if their main pokemon level is low
scale = 1 - min(trainer_level * (1/12), 1)
Copy link
Owner

Choose a reason for hiding this comment

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

should level thresholds be maintained for mainpkmn lvl independent of trainer lvl?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think better to take both - The Idea - to not get a legendary with like a simple Caterpie level 5

Copy link
Collaborator

Choose a reason for hiding this comment

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

We should add comments tho to this function (even if it is simple, we should always make sure to add proper PEP8 Documentation Functions)

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