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 animation keyframes being skipped when played backwards #81452

Merged
merged 1 commit into from
Sep 8, 2023

Conversation

tcoxon
Copy link
Contributor

@tcoxon tcoxon commented Sep 8, 2023

Fixes #57271.

Currently, if an animation playing backwards passes by multiple keyframes in a single frame, only the first of the keyframes (last in forwards-order) takes effect. The others are skipped. The cause appears to be a simple typo.

@akien-mga akien-mga added bug cherrypick:3.x Considered for cherry-picking into a future 3.x release topic:animation cherrypick:3.5 Considered for cherry-picking into a future 3.5.x release cherrypick:4.0 cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release labels Sep 8, 2023
@akien-mga akien-mga added this to the 4.2 milestone Sep 8, 2023
Copy link
Member

@akien-mga akien-mga left a comment

Choose a reason for hiding this comment

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

Definitely looks like a typo, so if you've confirmed this fixes the bug, it should be good.

@akien-mga akien-mga merged commit fb4edf5 into godotengine:master Sep 8, 2023
15 checks passed
@akien-mga
Copy link
Member

Thanks!

@akien-mga
Copy link
Member

Doesn't seem cherry-pickable to 3.x, this code doesn't exist there. So the 3.x bug might have a different cause.

@akien-mga akien-mga removed cherrypick:3.x Considered for cherry-picking into a future 3.x release cherrypick:3.5 Considered for cherry-picking into a future 3.5.x release labels Sep 24, 2023
@tcoxon
Copy link
Contributor Author

tcoxon commented Sep 24, 2023

I agree, the cause does seem different. I'm not able to make a working minimal reproducer for 3.x

@YuriSizov YuriSizov removed the cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release label Oct 24, 2023
@YuriSizov
Copy link
Contributor

Cherry-picked for 4.1.3.

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.

Backwards animations with Engine.time_scale set skip the final (first) keyframe
3 participants