Skip to content

Conversation

@bdach
Copy link
Collaborator

@bdach bdach commented Oct 6, 2025

  • Pending diffcalc runs in order to estimate possible impact of this change on all ranked beatmaps

@bdach bdach self-assigned this Oct 6, 2025
@bdach bdach added the area:beatmap parsing .osu file format parsing label Oct 6, 2025
@bdach bdach moved this from Next up to In Progress in osu! untitled project Oct 6, 2025
@bdach

This comment was marked as outdated.

@bdach

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@bdach
Copy link
Collaborator Author

bdach commented Oct 6, 2025

!diffcalc
RULESET=osu
NO_ASPIRE=0
GENERATORS=sr

@bdach
Copy link
Collaborator Author

bdach commented Oct 6, 2025

!diffcalc
RULESET=catch
NO_ASPIRE=0
GENERATORS=sr

@github-actions
Copy link

github-actions bot commented Oct 6, 2025

@github-actions
Copy link

github-actions bot commented Oct 6, 2025

@peppy
Copy link
Member

peppy commented Oct 7, 2025

Spreadsheet looks clean.

@bdach
Copy link
Collaborator Author

bdach commented Oct 7, 2025

It does, which makes me a little suspicious given that I'd expect to see at least the loved beatmap that sparked this in there.

I'll run one more and see if anything changes:

!diffcalc
RULESET=osu
NO_ASPIRE=0
GENERATORS=score

@github-actions
Copy link

github-actions bot commented Oct 7, 2025

@bdach
Copy link
Collaborator Author

bdach commented Oct 7, 2025

Yeah okay. I'm much happier with that last sheet. Other than the beatmap that this is targeting it also shows this beatmap as affected and, let's just say, I have run into this mapper before, and they're definitely... uhhhhhh... a fan of testing the game limits and taking creative liberties with those? is the polite way to put it? And that map does feature several zero-length sliders with non-zero repeat counts.

No idea why the "score losses" tab is showing a bunch of rows with score losses of zero, but I also don't think I am going to worry about that too much. Undrafting.

@bdach bdach marked this pull request as ready for review October 7, 2025 12:12
@bdach bdach requested review from peppy and smoogipoo October 7, 2025 12:13
@bdach bdach moved this from In Progress to Pending Review in osu! untitled project Oct 7, 2025
@smoogipoo smoogipoo merged commit 1a522a1 into ppy:master Oct 8, 2025
7 of 9 checks passed
@github-project-automation github-project-automation bot moved this from Pending Review to Done in osu! untitled project Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:beatmap parsing .osu file format parsing size/S

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants