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

TurboJpeg current implementation only talks RGB. Also using Turbojpeg should be an option, not a "library is there, we use it" #725

Open
DiegoPino opened this issue Jan 29, 2025 · 0 comments

Comments

@DiegoPino
Copy link
Contributor

What?

If you look at the TurboJpeg processor code, you will notice there is no Colorspace detection and there is always an assumption source is RGB. That means turbojpeg fails on CMYK. Java2D does not.

Also, turbojpeg is used as the default output processor when the libraries are present, making configuration changes (e.g swapping the JPEG processor for one that is known to work with CMYK) impossible. I think a more proper way would be to either check if JPEG is already mapped to TurboJpeg or have an extra setting that allows the output processing of e.g /default.jpeg to a config choice

@DiegoPino DiegoPino added this to the 6.0 milestone Jan 29, 2025
@DiegoPino DiegoPino self-assigned this Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant