Skip to content

Sample Transform from an AnimationClip#13778

Closed
mintlu8 wants to merge 3 commits into
bevyengine:mainfrom
mintlu8:anim_sample
Closed

Sample Transform from an AnimationClip#13778
mintlu8 wants to merge 3 commits into
bevyengine:mainfrom
mintlu8:anim_sample

Conversation

@mintlu8

@mintlu8 mintlu8 commented Jun 9, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor

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

mweatherley commented Oct 5, 2024

Copy link
Copy Markdown
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

atornity commented Oct 5, 2024

Copy link
Copy Markdown
Contributor

is there an issue for this?

@mweatherley

mweatherley commented Oct 5, 2024

Copy link
Copy Markdown
Contributor

is there an issue for this?

See #15664 :)

@atornity

atornity commented Oct 5, 2024

Copy link
Copy Markdown
Contributor

See #15664 :)

thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@mintlu8 @mweatherley @atornity @alice-i-cecile