-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Statistical accuracy PP and difficulty scaling for the osu!taiko ruleset #20963
Merged
Merged
Changes from all commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
442e68a
Implement taiko deviation estimation
Natelytle d5b06ae
Fix difficultyvalue acc scaling
Natelytle 607a006
oops
Natelytle 87cba2d
Slight adjustments
Natelytle 7d3338a
LTCA Balancing pass
Natelytle af919a6
harshen deviation scaling
Natelytle 2940d18
Fix formatting
Natelytle 883790c
Return null instead of infinity
Natelytle 01c79d8
remove other infinity reference
Natelytle 7403c1c
Return null for greatprobability >= 1
Natelytle 16301f0
Fix low end accuracy, buff high end
Natelytle 37c21cd
fix formatting
Natelytle 2ba1634
account for low acc FC deviation
Natelytle 0e4e92b
totalvalue
Natelytle b579af6
fix dt
Natelytle e3ef180
fixes
Natelytle 7b5373a
add comments
Natelytle 34533e5
Merge branch 'master' into taikostatacc
smoogipoo 6a27206
bugfix + tests
Natelytle 2b74c4e
tests return a greathitwindow of 0, add check
Natelytle 45e8d18
fix extremely low OD breaking deviation calc
Natelytle 334f60f
Reformat everything to be simpler
Natelytle d5ac73e
Merge remote-tracking branch 'osumaster/master' into taikostatacc
Natelytle adf1618
Change accuracy scaling
Natelytle 858afcd
Pass OK hit window as a separate difficulty attribute, fix erfc appro…
Natelytle 9aa11e0
update desmos
Natelytle 7ee9101
Merge remote-tracking branch 'osumaster/master' into taikostatacc
Natelytle 31c8cf0
Buff accuracy scaling
Natelytle 4de0246
Make comments more professional
Natelytle faddc4f
Merge remote-tracking branch 'osumaster/master' into taikostatacc
Natelytle e569420
Serialize ok hit window attribute to db
Natelytle 5f0020b
Reduce accuracy scaling
Natelytle 3a609c9
Merge branch 'master' into taikostatacc
smoogipoo 8a26cda
Merge master
Natelytle caba051
Compute the upper bound on deviation with a 99% confidence interval
Natelytle 6ddb2b7
Include misses in the great window deviation calc
Natelytle 5370595
Fix comment
Natelytle a9b3416
Remove MathNet.Numerics dependency
Natelytle 1714567
Save deviation calculations to variables
Natelytle f8f18b6
Fix naming convention
Natelytle 2fb22f1
Move the return value for deviation below the local functions
Natelytle 84d6467
Merge branch 'master' into taikostatacc
bdach File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since the changes here already account for hit windows properly, and does not concern SV, why does hard rock specifically need to be buffed here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was done in balancing, as mid-range accuracy with HR was pretty underweighted (4x 100 on the limit does not exist HR was worth 20pp less than a HD SS). Though @Natelytle could the model be fit for greater accuracy leniency on super high OD's?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think you can without increasing complexity and decreasing estimation accuracy, I think a HR multiplier buff is a better direction if HR in particular is underweight