Skip to content

Remove DrainRate difficulty attribute#35817

Merged
tsunyoku merged 1 commit into
ppy:pp-devfrom
Givikap120:remove-drain-attribute
Nov 29, 2025
Merged

Remove DrainRate difficulty attribute#35817
tsunyoku merged 1 commit into
ppy:pp-devfrom
Givikap120:remove-drain-attribute

Conversation

@Givikap120

Copy link
Copy Markdown
Contributor

I don't know how this was missed when removing AR and OD attributes, but here.
It wasn't stored in the database (it was local-only attribute), but I think it's weird why it's here when AR and OD aren't.

@stanriders stanriders left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think it got lost in the middle of the ar/od->diffcalc refactor because of the BL? Not sure

@stanriders stanriders requested a review from tsunyoku November 26, 2025 21:40
@tsunyoku

Copy link
Copy Markdown
Member

I was mainly concerned with reducing database storage and since this one isn't databased I probably just left it.

This also works though.

@tsunyoku tsunyoku merged commit 1b230b3 into ppy:pp-dev Nov 29, 2025
5 of 9 checks passed
@github-project-automation github-project-automation Bot moved this from Pending Review to Pending Deploy in Difficulty calculation changes Nov 29, 2025
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.

4 participants