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

FrameAnimation Rate #32

Closed
gyllen opened this issue Jan 27, 2016 · 5 comments
Closed

FrameAnimation Rate #32

gyllen opened this issue Jan 27, 2016 · 5 comments
Assignees
Labels

Comments

@gyllen
Copy link
Member

gyllen commented Jan 27, 2016

Not sure this is a bug, but Im running a screen thats on 30hz, this causes my animations to be half the speed of what you probably see them in.

@oscarlorentzon
Copy link
Member

There are a number of different ways to handle movement based on the frames and elapsed time. Examples are:

  • Transitions per frame
  • Transitions per second
  • Meters per frame
  • Meters per second

Right now we use Transitions per frame, i.e. when the frame rate is 30 frames per second the speed is half of the speed for a frame rate of 60 frames per second.

For different situations it can even be a good idea to use different movement schemes.

Changing to transitions per second would handle the situation with different frame rates.

@gyllen
Copy link
Member Author

gyllen commented Jan 27, 2016

Yeah its a sticky situation. For now a solution would be to adjust animationSpeed according to FrameAnimation speed. Im not sure if you can get that speed though. But for me animationSpeed should be double yours. Thats way we still keep smoothness on 60hz but we keep up speed on 30hz.

@oscarlorentzon
Copy link
Member

Exactly, transitions per second will handle time delta between frames and update animationSpeed accordingly.

For situations where the time delta between two frames is large there could be a cap on the animationSpeed.

@oscarlorentzon
Copy link
Member

@gyllen Do you consider this issue as fixed after the commit below?

8c48050

@gyllen
Copy link
Member Author

gyllen commented Mar 31, 2016

Fixed

@gyllen gyllen closed this as completed Mar 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants