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

mDCv and cLLi are now mDCV and cLLI #5

Open
svgeesus opened this issue Jan 6, 2025 · 3 comments
Open

mDCv and cLLi are now mDCV and cLLI #5

svgeesus opened this issue Jan 6, 2025 · 3 comments

Comments

@svgeesus
Copy link

svgeesus commented Jan 6, 2025

Because these chunks are unsafe-to-copy, the chunk names were changed:

No change to the contents of these chunks.

@ProgramMax
Copy link

I see this support cICP from Third Edition. But I don't think it support mDCV or cLLI yet, right?
So to clarify, I think the request isn't to catch up to the renaming--it's to preemptively add them as mDCV and cLLI instead of mDCv and cLLi when they do get implemented.

@tleguern
Copy link
Owner

tleguern commented Jan 7, 2025

Hello. Chunks mDCV and cLLI have never been implemented regardless of capitalisation. I never encountered a file containing either of them yet and frankly don't really understand what they do.

@ProgramMax
Copy link

I'm not a color expert so I'll try to answer to the best of my knowledge but take what I say with a grain of salt.

mDCV and cLLI provide information for the final display to apply tone mapping.
As an example, the image doesn't know the final display's capabilities. So maybe the image stores colors and brightness that the display can't handle. Tone mapping allows the display to bring things within its capabilities while still preserving important details for its intended viewing.

Like if we just darken all colors, maybe an important clue becomes too dim. So a flat or linear dark won't work. Maybe we need a curve or steps. The display knows how much processing power it has, its capabilities, and this content information -- thus what tone mapping algorithm is the best fit.

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

No branches or pull requests

3 participants