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

cjpegli ignores sRGB ICC profiles #3549

Open
DAPTYX opened this issue May 2, 2024 · 2 comments
Open

cjpegli ignores sRGB ICC profiles #3549

DAPTYX opened this issue May 2, 2024 · 2 comments
Labels
jpegli Related to jpegli, also cjpegli and djpegli unrelated to 1.0 Things that need not be done before the 1.0 version milestone

Comments

@DAPTYX
Copy link

DAPTYX commented May 2, 2024

Why does cjpegli ignore sRGB ICC profiles? Is this intended?
Works with all other profiles, but all sRGB variants are missing after conversion.

@mo271 mo271 added unrelated to 1.0 Things that need not be done before the 1.0 version milestone jpegli Related to jpegli, also cjpegli and djpegli labels May 2, 2024
@j7nj7n
Copy link

j7nj7n commented May 3, 2024

sRGB should be default for JPEG and doesn't need to be explicitly declared. It saves 1 to 3 kB of data, and avoids simple programs loading the image slowly by applying a sRGB->sRGB conversion. Maybe there should be an optional switch. "Compacted" alternate sRGB profiles still trigger a conversion in the image viewer I use because it doesn't know they are practically the same.

@DAPTYX
Copy link
Author

DAPTYX commented May 3, 2024

Color managed environments use profiles, the default might be sRGB or something else.
What's strange is that the most basic profiles are not only treated as unnecessary, they are technically forbidden. If users don't want profiles to be attached, they don't add them in the first place - personally, I don't see any other way to think about it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
jpegli Related to jpegli, also cjpegli and djpegli unrelated to 1.0 Things that need not be done before the 1.0 version milestone
Projects
None yet
Development

No branches or pull requests

3 participants