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

Default to 30 animation fps for better quality. #51341

Merged
merged 1 commit into from
Nov 9, 2021

Conversation

fire
Copy link
Member

@fire fire commented Aug 6, 2021

Reduz mentioned that he was working on animation streaming for Godot Engine. It was unclear when his animation streaming support will be complete and animation streaming is an optimization, so this [animation streaming] feature is unrelated to the bug-fix of animations defaulting to the wrong fps and skipping transforms by under-sampling.

@fire fire requested a review from a team as a code owner August 6, 2021 23:06
@Calinou Calinou added this to the 4.0 milestone Aug 6, 2021
@akien-mga
Copy link
Member

akien-mga commented Aug 7, 2021

It would be nice to amend the commit message to refer explicitly to animation fps.
Otherwise you're going to give users a heart attack :)

@fire fire changed the title Default to 30 fps for better quality. Default to 30 animation fps for better quality. Aug 7, 2021
@reduz
Copy link
Member

reduz commented Aug 30, 2021

I really want to implement animation compression and streaming so 30fps can be made default, but this is a lot of work. The problem with 30fps animation is that right now they take a lot of space.

@Calinou
Copy link
Member

Calinou commented Oct 21, 2021

godotengine/godot-proposals#430 would likely be a better solution for animations designed with a lower framerate. However, this PR can work as a stopgap since many animators use 30 FPS as a baseline.

@akien-mga akien-mga requested a review from reduz October 22, 2021 17:34
@fire
Copy link
Member Author

fire commented Nov 9, 2021

@reduz animation compression was implemented, as discussed in rocket chat, can you approve on Github?

@fire fire merged commit 081c1ce into godotengine:master Nov 9, 2021
@fire
Copy link
Member Author

fire commented Nov 9, 2021

Thanks!

@fire fire deleted the fps-30 branch November 9, 2021 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants