Skip to content

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

Closed
mockersf wants to merge 1 commit intobevyengine:mainfrom
mockersf:restore-model-between-animations
Closed

[Merged by Bors] - scene viewer improvements: animation reset#4420
mockersf wants to merge 1 commit intobevyengine:mainfrom
mockersf:restore-model-between-animations

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
@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