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 the character select cursor lerp for lower framerates #3507

Merged
merged 1 commit into from
Oct 4, 2024

Conversation

ninjamuffin99
Copy link
Member

fixes #3505

Updating the main cursor to use the smoothLerp() instead of coolLerp() seems to fix it, as the rest of the squares lerp towards the main yellow cursor.

I believe this bug is caused from the lerp overshooting and needing to recorrect itself using the old lerp method

@ninjamuffin99 ninjamuffin99 changed the base branch from main to develop October 1, 2024 05:05
@trayfellow
Copy link

Maybe all instances of the deprecated coolLerp function in the game should just be replaced with smoothLerp? Or would that still cause a bunch of issues

@ninjamuffin99
Copy link
Member Author

coolLerp interpolates in a different way than smoothLerp, smoothLerp interpolates in a given time, so for each instance you'd need to tweak some values to be able to match the lerp visually

@ninjamuffin99 ninjamuffin99 force-pushed the develop branch 2 times, most recently from e0b1b01 to 410cfe9 Compare October 4, 2024 01:25
@ninjamuffin99 ninjamuffin99 force-pushed the ninjamuffin99/funk-1023 branch from 1373f47 to 11d3420 Compare October 4, 2024 16:32
@ninjamuffin99 ninjamuffin99 merged commit ab5bda3 into develop Oct 4, 2024
@EliteMasterEric EliteMasterEric added this to the 0.5.2 milestone Oct 12, 2024
@Hundrec Hundrec added type: minor bug Involves a minor bug or issue. status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. size: small A small pull request with 10 or fewer changes. labels Jan 22, 2025
@AbnormalPoof AbnormalPoof added the pr: haxe PR modifies game code. label Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: haxe PR modifies game code. size: small A small pull request with 10 or fewer changes. status: accepted PR was approved for contribution. If it's not already merged, it may be merged on a private branch. type: minor bug Involves a minor bug or issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug Report: Character select square completely breaks at framerates below 60FPS
5 participants