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

Fixed Variant::zero() to explicitly return a value of zero #59490

Closed
wants to merge 1 commit into from

Conversation

TokageItLab
Copy link
Member

@TokageItLab TokageItLab commented Mar 24, 2022

Follow up to #57675.

Fixed #59474.

It must explicitly return a value of zero for Variant::blend(). However, since Quaternion will does slerp without expmap in Variant::blend(), so the value of w must be 1.

Another solution would be to implement a function that performs subtraction, such as Variant::sub(), but that would be a bit exaggerated.

@TokageItLab TokageItLab added this to the 4.0 milestone Mar 24, 2022
@TokageItLab TokageItLab requested a review from a team as a code owner March 24, 2022 21:31
@TokageItLab
Copy link
Member Author

Superseded by #60093.

@TokageItLab TokageItLab removed this from the 4.0 milestone Apr 13, 2022
@TokageItLab TokageItLab deleted the fix-variant-zero branch May 2, 2022 19:38
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.

4.0 Alpha 5 regression: Color blending with alpha channel by AnimationTree is broken
1 participant