Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Comments

osu!taiko new miss penalty using consistency factor#33409

Merged
tsunyoku merged 4 commits intoppy:pp-devfrom
buyaspacecube:miss-penalty-4-merge
Jun 8, 2025
Merged

osu!taiko new miss penalty using consistency factor#33409
tsunyoku merged 4 commits intoppy:pp-devfrom
buyaspacecube:miss-penalty-4-merge

Conversation

@buyaspacecube
Copy link
Member

These changes rework effectiveMissCount to be based on a map's total hits and consistency factor, as well as punishing said effective misses more on maps with less total hits and vice versa. Not a huge amount of effort has been put into balancing the numbers, as we plan on doing a full balancing pass once everything we want before next deploy is implemented

@buyaspacecube buyaspacecube requested a review from Lawtrohux June 5, 2025 11:00
Copy link
Member

@Lawtrohux Lawtrohux left a comment

Choose a reason for hiding this comment

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

lgtm

@stanriders stanriders moved this from Pending Review to Pending Maintainers Review in Difficulty calculation changes Jun 6, 2025
@tsunyoku tsunyoku enabled auto-merge (squash) June 8, 2025 22:42
@tsunyoku tsunyoku moved this from Pending Maintainers Review to Pending Merge in Difficulty calculation changes Jun 8, 2025
@tsunyoku tsunyoku merged commit 5df41c0 into ppy:pp-dev Jun 8, 2025
4 of 8 checks passed
@github-project-automation github-project-automation bot moved this from Pending Merge to Pending Deploy in Difficulty calculation changes Jun 8, 2025
@buyaspacecube buyaspacecube deleted the miss-penalty-4-merge branch August 2, 2025 11:31
Finadoggie pushed a commit to Finadoggie/osu that referenced this pull request Aug 18, 2025
* New formulas for effective miss count and penalty

* More elaborate comments

* More comment stuff
@stanriders stanriders moved this from Pending Deploy to Deployed in Difficulty calculation changes Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Deployed

Development

Successfully merging this pull request may close these issues.

3 participants