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

Skip palette generation step when converting to gif #85

Merged
merged 1 commit into from
May 20, 2018
Merged

Conversation

brarcher
Copy link
Owner

@brarcher brarcher commented May 20, 2018

GIF files only support 256 colors. Previously a pass over the input
video was taken to develop a good color palette before converting
to gif. However, it was found that on a recent FFmpeg version that
the palette generation step was taking a significant amount of
memory and time to complete. For non-trivial videos it was not
possible to create a gif.

This change removes the palette generation step. Instead, a default
palette that FFmpeg defines is used. This will likely
cause the color quality of the output gif to decrease, however
it should be much quicker to generate the gif.

If the quality of the generated gif files is not that great, here are some options
to improving the quality:
http://blog.pkh.me/p/21-high-quality-gif-with-ffmpeg.html

#67

GIF files only support 256 colors. Previously a pass over the input
video was taken to develop a good color palette before converting
to gif. However, it was found that on a recent FFmpeg version that
the palette generation step was taking a significant amount of
memory and time to complete. For non-trivial videos it was not
possible to create a gif.

This change removes the palette generation step. This will likely
cause the color quality of the output gif to decrease, however
it should be much quicker to generate the gif.
@brarcher brarcher merged commit c4c9254 into master May 20, 2018
@brarcher brarcher deleted the simpler-gif branch May 20, 2018 03:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant