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

Changes to the calculation of speed and flow aim difficulty #28846

Closed
wants to merge 19 commits into from

Conversation

TextAdventurer12
Copy link
Contributor

@TextAdventurer12 TextAdventurer12 commented Jul 13, 2024

This contains a handful of changes:

  • an attempt to stop the double-dipping of spaced streams into speed and aim pp by reducing aim pp if spacing is less than the single spacing bonus
  • a nerf to the speed bonus
  • an angle bonus for distance in speed calculation
  • an increase to strain decay base for speed, resulting in a buff to longer streams

Values when combined with the other difficulty calculation changes (#28843 + #28844 + #28845 + #28846) can be seen here (onion access only)

@TextAdventurer12 TextAdventurer12 marked this pull request as ready for review July 15, 2024 09:39
@smoogipoo
Copy link
Contributor

!diffcalc

Copy link

github-actions bot commented Jul 16, 2024

@smoogipoo
Copy link
Contributor

!diffcalc

Copy link

github-actions bot commented Jul 18, 2024

Difficulty calculation failed: https://github.com/ppy/osu/actions/runs/9985735036

@smoogipoo
Copy link
Contributor

!diffcalc

Copy link

github-actions bot commented Jul 19, 2024

Difficulty calculation failed: https://github.com/ppy/osu/actions/runs/10002689393

@smoogipoo
Copy link
Contributor

!diffcalc

Copy link

github-actions bot commented Jul 19, 2024

Difficulty calculation failed: https://github.com/ppy/osu/actions/runs/10004243543

@TextAdventurer12 TextAdventurer12 marked this pull request as draft August 24, 2024 10:03
@TextAdventurer12
Copy link
Contributor Author

moved back to draft while I work through some feedback

@TextAdventurer12
Copy link
Contributor Author

closing this because it underwent changes on another branch alongside some other changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Pending Deploy
Development

Successfully merging this pull request may close these issues.

5 participants