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

Fix slider paths being extended even when the last two points are equal #15303

Merged
merged 4 commits into from
Oct 26, 2021

Conversation

peppy
Copy link
Member

@peppy peppy commented Oct 26, 2021

I'm not sure if this is a valid solution. We likely don't want this in SliderPath, but that said, it shouldn't affect any normal scenario (where a user is creating a beatmap in the lazer editor).

PRing this mainly for discussion purposes. If this is considered okay then I'll add test coverage.

Closes #15109.

@smoogipoo
Copy link
Contributor

Apart from test failures, I think this is fine.

@smoogipoo smoogipoo merged commit 61c8c96 into ppy:master Oct 26, 2021
@peppy peppy deleted the fix-slider-path-extension-incorrectness branch October 26, 2021 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Certain slider lengths aren't equal to stable lengths
2 participants