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

[enhancement] TIFF-Support #643

Closed
JoKalliauer opened this issue Sep 26, 2024 · 3 comments
Closed

[enhancement] TIFF-Support #643

JoKalliauer opened this issue Sep 26, 2024 · 3 comments
Labels
I-Minor Issues that would be nice to have fixed, but are the lowest priority T-Feature Requests for a new feature to be added

Comments

@JoKalliauer
Copy link

Is support of other formats, such as tiff, jpeg, webp desired/planed or out of scope?

@ace-dent
Copy link

My 2¢:
The executable is already quite large. I'd prefer oxipng to remain very focussed as a command line tool, so suggest conversion is done with other utility programs.

@AlexTMjugador
Copy link
Collaborator

I'd say support for optimizing other image formats is clearly out of scope.

Support for reading other image formats and converting them to optimized PNGs could be more appropriate (see #480 for related discussion), but given the PNG format lossless nature and the ability to tune its filter selecting and Deflate encoding parameters for speed over compression size, it is practical to use alternative encoders for faster-than-usual PNG encoding, reducing the need for OxiPNG to support more input formats from a user perspective. Furthermore, Rust library and application developers can already use the internal raw image API to decode images from any format they prefer. Therefore, I agree with @ace-dent's perspective that it seems more sensible to keep OxiPNG focused on handling PNGs.

@AlexTMjugador AlexTMjugador added T-Feature Requests for a new feature to be added I-Minor Issues that would be nice to have fixed, but are the lowest priority labels Sep 28, 2024
@andrews05
Copy link
Collaborator

Indeed, since there is already an open issue for other input formats and other output formats is out of scope, I think we can safely close this issue.

TIFF: Maybe imagemagick?
JPEG: jpeg-recompress, jpegli
WebP: Unsure, probably tuning the parameters to cwebp is your best bet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I-Minor Issues that would be nice to have fixed, but are the lowest priority T-Feature Requests for a new feature to be added
Projects
None yet
Development

No branches or pull requests

4 participants