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

Add a CallbackModeDiscrete property to AnimationMixer to handle the case of blending Continuous and Discrete tracks #86629

Merged
merged 1 commit into from
Feb 17, 2024

Conversation

TokageItLab
Copy link
Member

@TokageItLab TokageItLab commented Dec 30, 2023

@TokageItLab
Copy link
Member Author

TokageItLab commented Feb 10, 2024

Fixed a bug where INTERPOLATION_NEAREST did not handle reverse playback correctly (the bug existed before this PR). This ensures consistency in playback when converting from Discrete.

However, the old Nearest interpolation is not unreasonable, so for now this is only used when converted from Discrete by Force Continuous option. If necessary, we can add it to Animation::InterpolationType can be added later (as like INTERPOLATION_NEAREST_BIDIRECTIONAL).

@AThousandShips AThousandShips removed request for a team February 13, 2024 10:47
@TokageItLab TokageItLab requested review from fire and removed request for a team February 13, 2024 22:42
Copy link
Contributor

@lyuma lyuma left a comment

Choose a reason for hiding this comment

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

After Tokage explained it to me, I understand what this is fixing and it makes sense.

I do think users are going to be confused, so I'm suggesting some documentation changes.

doc/classes/AnimationMixer.xml Outdated Show resolved Hide resolved
doc/classes/AnimationMixer.xml Outdated Show resolved Hide resolved
doc/classes/AnimationMixer.xml Outdated Show resolved Hide resolved
doc/classes/AnimationMixer.xml Outdated Show resolved Hide resolved
misc/extension_api_validation/4.2-stable.expected Outdated Show resolved Hide resolved
scene/resources/animation.h Outdated Show resolved Hide resolved
@akien-mga akien-mga merged commit e31b253 into godotengine:master Feb 17, 2024
16 checks passed
@akien-mga
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
4 participants