Skip to content

Fix animation transitions affecting other entities #18572

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

Merged

Conversation

greeble-dev
Copy link
Contributor

@greeble-dev greeble-dev commented Mar 27, 2025

Objective

Fix #18557.

Solution

As described in the bug, remaining_weight should have been inside the loop.

Testing

Locally changed the animated_mesh_control example to spawn multiple meshes and play different transitions.

Copy link
Contributor

@kristoff3r kristoff3r left a comment

Choose a reason for hiding this comment

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

It would be great with a regression test as well

@greeble-dev
Copy link
Contributor Author

I've made a test but put it in a separate PR: #18574. I'm assuming this PR is best kept small and uncontroversial so it can get into 0.16.

@mockersf mockersf added this pull request to the merge queue Mar 27, 2025
@mockersf mockersf added this to the 0.16 milestone Mar 27, 2025
Merged via the queue into bevyengine:main with commit a7e6578 Mar 27, 2025
33 checks passed
mockersf pushed a commit that referenced this pull request Mar 27, 2025
## Objective

 Fix #18557.

## Solution

As described in the bug, `remaining_weight` should have been inside the
loop.

## Testing

Locally changed the `animated_mesh_control` example to spawn multiple
meshes and play different transitions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Animation transitions don't apply weights correctly
3 participants