-
-
Notifications
You must be signed in to change notification settings - Fork 292
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
Allow 60 FPS? #161
Comments
There are a couple problems:
|
What currently happens when a user sets it to, for example, 22 FPS? We have a slider in Gifski.app that can be set to any integer between 5 and 30. Does gifski-lib round it internally?
Does gifski-lib do this?
Any source/link about this? I wonder if it also applies to 30 FPS, or only much higher values. It seems ffmpeg does indeed does this average trick to get an arbitrary FPS: |
Currently the app rounds the delay, so it's not great: Line 190 in 5880fd3
The CLI version uses the alternating frame timing trick. I plan to change the C API to take That's the original article I had in mind (RIP Tumlbr): https://nullsleep.tumblr.com/post/16524517190/animated-gif-minimum-frame-delay-browser Here's explicit info from (no longer relevant) browser vendor: https://blogs.msdn.microsoft.com/ieinternals/2010/06/07/trivia-animated-gif-timing/ |
Hi Sindre Sorhus, Request you please please please add an option of 60 FPS. There are very few apps available but all of them are crap. Big crap. If you will add this feature then it would be awesome. You might be getting few requests for 60 FPS but please add this option. Keep it up the great work! Much Love, |
The actual result will be 50 FPS, but I doubt the user will notice. See: #161
from a physiological perspective film is 24 fps about he frame rate the brain processes stuff as motion so i say 25fps is a good default |
I've gotten a couple of requests for 60 FPS. We currently limit it to 30 FPS. I do realize it doesn't even usually make sense with 30 FPS for GIF, but their use-case is very short clips to upload to https://dribbble.com.
Thoughts?
The text was updated successfully, but these errors were encountered: