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

add support for webp encoding #393

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

add support for webp encoding #393

wants to merge 2 commits into from

Conversation

willnorris
Copy link
Owner

This is an experimental branch that adds webp encoding. In my very limited testing, this seems to add quite a lot of visual artifacts at 90% compress, and results in larger files sizes at lossless compress. Now granted, that was only with a sample size of 1 using https://octodex.github.com/images/codercat.jpg. I'll try more testing later, but wanted to get this posted for others to take a look at and try out.

Updates #388

@robgordon89
Copy link

@willnorris 👋 firstly thanks for this amazing project, I have tested this it works really well the issue I had with WebP was transparency and this seems to have fixed that, I also wanted to say that at 90% quality its very distorted but at 100% it seems to be 4x larger 🤔

Might be better to default to 100% quality

In any case this does add better support IMO even with the increased size, i tried the latest go-libwebp 1.8.0 without any decrease in final image size.

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.

2 participants