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

Make offset adjust more lenient #31645

Merged
merged 3 commits into from
Jan 24, 2025
Merged

Conversation

peppy
Copy link
Member

@peppy peppy commented Jan 24, 2025

Noticed in passing that we are a bit too harsh, basing the restrictions on time=0 instead of gameplay start time (see stable reference).

Of note, it's still a bit more strict due to the 2 second offset to GameplayStartTime in lazer. Not sure this is too important, but if we want to match 1:1 we can either change the allowance to 12 seconds or source the GameplayStartTime directly from the beatmap, I guess.

@bdach bdach self-requested a review January 24, 2025 10:08
@bdach bdach self-requested a review January 24, 2025 12:52
@bdach bdach merged commit 7845c5c into ppy:master Jan 24, 2025
10 checks passed
@peppy peppy deleted the offset-adjust-more-lenience branch January 28, 2025 13:10
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.

2 participants