Skip to content

Conversation

@digitaltvguy
Copy link
Contributor

to replace current definition for "reference image" which only described Full Range Quantization.

to replace current definition for "reference image" which only described Full Range Quantization.
@digitaltvguy digitaltvguy linked an issue Sep 30, 2024 that may be closed by this pull request
Copy link
Contributor

@svgeesus svgeesus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this proposed change needs Working Group discussion before being merged.

  • strange mention of YCbCr, which PNG does not support
  • replaces 2^sampledepth with 16

@ProgramMax

Copy link
Member

@ProgramMax ProgramMax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed with CL.
I added it to the discussion topics.
We should also mark it as informative.

<th>White</th>
</tr>

<table border="1">
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this line can be removed. It accidentally adds an extra table inside the other table. Notice, it also never gets closed.

@ProgramMax
Copy link
Member

Also, I notice this is in § 4.2 Images - Reference image. Perhaps that section should instead be changed to say something like "1 to 16 bits, although not all bits might be used. The sBIT chunk (link) confines the bit range, and the cICP chunk (link) might declare the image is narrow range."

Then the cICP chunk could have these example values instead of 4.2

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

Successfully merging this pull request may close these issues.

Narrow range and cICP

4 participants