Skip to content

Added table with 16bit full and narrow range values#472

Draft
digitaltvguy wants to merge 3 commits into
mainfrom
312-testing-cicp
Draft

Added table with 16bit full and narrow range values#472
digitaltvguy wants to merge 3 commits into
mainfrom
312-testing-cicp

Conversation

@digitaltvguy

Copy link
Copy Markdown
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

@svgeesus svgeesus left a comment

Copy link
Copy Markdown
Contributor

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

@ProgramMax ProgramMax left a comment

Copy link
Copy Markdown
Member

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.

Comment thread index.html
<th>White</th>
</tr>

<table border="1">

Copy link
Copy Markdown
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
Copy Markdown
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

@svgeesus svgeesus marked this pull request as draft November 19, 2025 06:55
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

3 participants