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

Rewrite of the colour skill & refactoring of difficulty calculation within osu!taiko #19566

Closed
wants to merge 60 commits into from

Conversation

Lawtrohux
Copy link
Member

@Lawtrohux Lawtrohux commented Aug 3, 2022

reopen of #19184.

This PR aims to get all the major refactoring out of the way, and with the addition of the rewritten colour skill. Apologies for the large size of the PR, as everything is much too intertwined to properly split this PR. This should be the last PR of its out of proportion scale.

Full rewrite on the colour skill based on information density. See here for a more detailed explanation.
Adjusted weight of each skill. Rhythm is intentionally weighted lower due to it not being worked on yet.
Changes to taiko difficulty hit object to incorporate evaluators.
Ported stamina and colour into stateless evaluators.
Changed stamina speed bonus to be 1 / interval based.
Removed separated ratings for final SR Calculation.
Moved locally combined difficulty (strain peaks) into the peaks skill, all other skills are now under this.
Changed locally combined difficulty to use 2-norm to combine stamina and colour, and a 1.5-norm to combine that with rhythm.
Addition of data directories, which will be used in future references per skill.
Addition of per skill pre-processing, to clean up clutter.


SR/PP sheet versus master prior to #19181 : https://docs.google.com/spreadsheets/d/1bSE9UGLG50U1r8Ih90A9kheGbPDrDx1sXtmNW9VCdIg/edit

As of smoogipoo@61d2b8e

vunyunt and others added 30 commits May 24, 2022 17:38
@smoogipoo
Copy link
Contributor

SR/PP sheet added to OP.

@vunyunt
Copy link
Contributor

vunyunt commented Aug 3, 2022

Closing this due to the Allow edits and access to secrets by maintainers option not being available when opened from an organisation. #19571 has been opened from my repository in its place. Changes should be identical.

@vunyunt vunyunt closed this Aug 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants