-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Account for alternating angles in reading angle factor #36466
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
base: pp-dev
Are you sure you want to change the base?
Conversation
|
This just feels way too targeted for my liking. This is pretty ugly in that it is littered with a bunch of conditions to satisfy with very specific constants, which I'm not really a fan of. |
That was intentional to make the nerf narrow-scoped for specific patterns, |
|
Feel free to post anything relevant here. |
|
The next check is for spacings being similar. As there's not a lot of point in checking repetitive alternating angles if spacings of the jumps are random and unpredictable.
The deltas here are much much larger than in previous check, and in some cases they're comparable with c-type nerf level (such as on the "Tatarigami" map (https://osu.ppy.sh/beatmapsets/724673#osu/1530206) |
|
The last check is for one of the angles being sharp and other being wide. Removing this check completely is not an option, because it completely ruins the whole algorithm: maps with many sharp angled jumps that are not really alternating are suffering major nerf, when alternating angle maps are barely touched because the difference between 180 degree and 150 degree is considered the same as between 0 degree and 30 degree, even though on practice it doesn't feels that way. Instead those are the deltas of limits being expanded to include much more sharp angled angles, as currently it mainly targets angles very close to 0 degrees.
As we could see the amount of affected maps is actually not that big, but ones that are affected - affected significantly. The 2 outlier maps here are 5150 (https://osu.ppy.sh/beatmapsets/812921#osu/1704984) and dreamin attraction (https://osu.ppy.sh/beatmapsets/2211043#osu/4683658). |
|
@tsunyoku that has to be all relevant info atm. |







Current state of rework is pretty much targeted C-type nerf, as this map gets absurd amount of reading pp, because angles are considered very unrepetitive. What results in this score being worth absurd 1.4k pp.
This PR is up to heavy discussion because it can be made much more general, touching more maps. So I wait on pp committee opinion on what of the parts can be removed.
Current checks for angle to be nerfed:
If pattern meets all the criteria - it would be considered repetitive. For now practically no map meets this criteria to significant amount except C-type.