-
-
Notifications
You must be signed in to change notification settings - Fork 315
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
Support writing WebP format #659
Comments
I'd like that too! 😀 |
@haraldk I'm sure you've seen https://github.com/sejda-pdf/webp-imageio |
@maxim5 Yes, I believe there are various ImageIO plugins that use the native libwebp for reading/writing through JNI. They probably perform better than our Java implementation, so if you can, use them! 😀 |
Yeah, don't have a lot of choices. That project seems to work, but hasn't updated |
I have no plans for going the JNI route with TwelveMonkeys. But I think having several implementations, with different strengths is only a good thing. 👍🏻 |
I have worked on that project for a bit. The painful part is to compile the lib and JNI bit together, for multiple platforms. Because it's native code, you need a different compiled binary for each combination of OS and cpu architecture. And you need a specific loader to load the right one. The issue with that lib is that it's not maintained, I tried to submit PRs to add support for more architectures, but it never went through. I ended up publishing it under my own org on maven central, and it's used in my open source application with some good success. I also use Twelvemonkeys for webp as a backup, since the app only needs to read webp, but never write it. |
Hi, is it possible to add support to write WEBP? |
@EricBlanquer It's absolutely possible. I don't have spare time to write it. That's why the issue is tagget with "sponsor needed". But anyone is free to write it and create a PR. I would happily accept it.😀 |
Describe the solution you'd like
As webp is a widely supported standard in browsers it should be possible to write in that format with this library to allow using it on server side for this use case.
The text was updated successfully, but these errors were encountered: