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

Warn for too large GIFs #220

Open
soshimee opened this issue Jan 15, 2021 · 7 comments
Open

Warn for too large GIFs #220

soshimee opened this issue Jan 15, 2021 · 7 comments

Comments

@soshimee
Copy link

I just checked out Gifski, then I realized it wasn't working, so I tried the CLI version. CLI version worked fine, but Mac App Store one could barely process 64x64 sized videos.
Maybe related to Big Sur update?

@kornelski
Copy link
Collaborator

kornelski commented Jan 15, 2021

Does it apply to all videos, or only videos in a specific format?

Can you use Activity Monitor to create a spindump report of the app while it's processing?

@soshimee
Copy link
Author

Dunno, can you give me some sample videos? I don't have that many videos so... The current video format that I'm using is mp4.

@fregante
Copy link

Just as another point: Today I tried converting a 12MB 720p MOV screencast on Big Sur and it suggested 4 minutes build time and >120MB output. I don't know if that's standard.

@kornelski
Copy link
Collaborator

kornelski commented Jan 19, 2021

Oh, that's totally expected. That's how the awful GIF format from 1989 works.

GIF was designed for "16p" animations. Throwing 720p content at it is pushing GIF to its limits, so expect it to be incredibly slow and unfathomably huge.

I suggest limiting HD-ish videos to very short clips, shorter than a second, or using very low resolution, like 128 pixels max.

Modern video codecs are many times faster and compress 10 or 20 times more efficiently than GIF, so expect the worst when going back in time.

@kornelski
Copy link
Collaborator

@sindresorhus Maybe the app could warn about creating overkill GIFs? Warn when resolution is >= 720p, and/or when the total size of all frames is huge.

@soshimee
Copy link
Author

soshimee commented Jan 20, 2021 via email

@sindresorhus
Copy link
Owner

sindresorhus commented Jan 22, 2021

@soshimee It did indeed get slower with Big Sur. Would you be able to share the 64x64 video with me? (You can send it privately too). Gifski was always slow with 720P video, but a 64x64 video should be no problem.

sindresorhus added a commit that referenced this issue Jan 22, 2021
@sindresorhus sindresorhus changed the title Mac app ridiculously slow Warn for too large GIFs Mar 14, 2021
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

No branches or pull requests

4 participants