Skip to content

Sample Transform from an AnimationClip #13778

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

Closed
wants to merge 3 commits into from

Conversation

mintlu8
Copy link
Contributor

@mintlu8 mintlu8 commented Jun 9, 2024

Objective

Allow users to obtain transform information from an animation without animating.

Solution

Added the ability to sample Transform from an AnimationClip at a given time.

Testing

TODO

Changelog

  • Added sample_transform_at to AnimationClip.
  • Added write_to_transform to VariableCurve.

@alice-i-cecile alice-i-cecile added C-Usability A targeted quality-of-life change that makes Bevy easier to use A-Animation Make things move and change over time S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged labels Jun 9, 2024
@mintlu8 mintlu8 changed the title Added the ability to sample Transform from an AnimationClip at a given time. Sample Transform from an AnimationClip Jun 9, 2024
@mweatherley
Copy link
Contributor

Closing this for now since, after #15434 and #15589, this probably needs to be revisited in a completely different way.

@mweatherley mweatherley closed this Oct 5, 2024
@atornity
Copy link
Contributor

atornity commented Oct 5, 2024

is there an issue for this?

@mweatherley
Copy link
Contributor

is there an issue for this?

See #15664 :)

@atornity
Copy link
Contributor

atornity commented Oct 5, 2024

See #15664 :)

thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Animation Make things move and change over time C-Usability A targeted quality-of-life change that makes Bevy easier to use S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants