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

[Merged by Bors] - scene viewer improvements: animation reset #4420

Closed

Conversation

mockersf
Copy link
Member

@mockersf mockersf commented Apr 4, 2022

Objective

  • Changing animation mid animation can leave the model not in its original position
  • The movement speed is fixed, no matter the size of the model

Solution

  • when changing animation, set it to its initial state and wait for one frame before changing the animation
  • when settings the camera controller, use the camera transform to know how far it is from the origin and use the distance for the speed

@github-actions github-actions bot added the S-Needs-Triage This issue needs to be labelled label Apr 4, 2022
@mockersf mockersf added C-Examples An addition or correction to our examples and removed S-Needs-Triage This issue needs to be labelled labels Apr 4, 2022
@alice-i-cecile alice-i-cecile added the A-Animation Make things move and change over time label Apr 4, 2022
@alice-i-cecile
Copy link
Member

Tested this out, I like the changes! The movement speed here is slower than before on the flight helmet model, but I'm a fan of that change.

@mockersf mockersf force-pushed the restore-model-between-animations branch from 9a95a61 to c03cac6 Compare April 4, 2022 23:27
examples/tools/scene_viewer.rs Outdated Show resolved Hide resolved
@mockersf mockersf force-pushed the restore-model-between-animations branch from c03cac6 to 98ee0e1 Compare April 5, 2022 23:07
@mockersf mockersf changed the title scene viewer improvements: animation reset and movement speed scene viewer improvements: animation reset Apr 5, 2022
@alice-i-cecile
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Apr 6, 2022
# Objective

- Changing animation mid animation can leave the model not in its original position
- ~~The movement speed is fixed, no matter the size of the model~~

## Solution

- when changing animation, set it to its initial state and wait for one frame before changing the animation
- ~~when settings the camera controller, use the camera transform to know how far it is from the origin and use the distance for the speed~~
@bors
Copy link
Contributor

bors bot commented Apr 6, 2022

Timed out.

@alice-i-cecile
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Apr 6, 2022
# Objective

- Changing animation mid animation can leave the model not in its original position
- ~~The movement speed is fixed, no matter the size of the model~~

## Solution

- when changing animation, set it to its initial state and wait for one frame before changing the animation
- ~~when settings the camera controller, use the camera transform to know how far it is from the origin and use the distance for the speed~~
@bors
Copy link
Contributor

bors bot commented Apr 6, 2022

This PR was included in a batch that timed out, it will be automatically retried

bors bot pushed a commit that referenced this pull request Apr 6, 2022
# Objective

- Changing animation mid animation can leave the model not in its original position
- ~~The movement speed is fixed, no matter the size of the model~~

## Solution

- when changing animation, set it to its initial state and wait for one frame before changing the animation
- ~~when settings the camera controller, use the camera transform to know how far it is from the origin and use the distance for the speed~~
@bors
Copy link
Contributor

bors bot commented Apr 6, 2022

This PR was included in a batch that timed out, it will be automatically retried

bors bot pushed a commit that referenced this pull request Apr 7, 2022
# Objective

- Changing animation mid animation can leave the model not in its original position
- ~~The movement speed is fixed, no matter the size of the model~~

## Solution

- when changing animation, set it to its initial state and wait for one frame before changing the animation
- ~~when settings the camera controller, use the camera transform to know how far it is from the origin and use the distance for the speed~~
@bors
Copy link
Contributor

bors bot commented Apr 7, 2022

Timed out.

@alice-i-cecile
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Apr 7, 2022
# Objective

- Changing animation mid animation can leave the model not in its original position
- ~~The movement speed is fixed, no matter the size of the model~~

## Solution

- when changing animation, set it to its initial state and wait for one frame before changing the animation
- ~~when settings the camera controller, use the camera transform to know how far it is from the origin and use the distance for the speed~~
@bors bors bot changed the title scene viewer improvements: animation reset [Merged by Bors] - scene viewer improvements: animation reset Apr 7, 2022
@bors bors bot closed this Apr 7, 2022
aevyrie pushed a commit to aevyrie/bevy that referenced this pull request Jun 7, 2022
# Objective

- Changing animation mid animation can leave the model not in its original position
- ~~The movement speed is fixed, no matter the size of the model~~

## Solution

- when changing animation, set it to its initial state and wait for one frame before changing the animation
- ~~when settings the camera controller, use the camera transform to know how far it is from the origin and use the distance for the speed~~
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
# Objective

- Changing animation mid animation can leave the model not in its original position
- ~~The movement speed is fixed, no matter the size of the model~~

## Solution

- when changing animation, set it to its initial state and wait for one frame before changing the animation
- ~~when settings the camera controller, use the camera transform to know how far it is from the origin and use the distance for the speed~~
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-Examples An addition or correction to our examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants