Skip to content

Addresses editorial feedback from Yannis, Maryla, and Thomas#61

Draft
podborski wants to merge 4 commits intomainfrom
additional_review
Draft

Addresses editorial feedback from Yannis, Maryla, and Thomas#61
podborski wants to merge 4 commits intomainfrom
additional_review

Conversation

@podborski
Copy link
Copy Markdown
Member

Done

  • reserved_zeros: Changed "Will be set to zeroes" to "Shall be set to zeroes"
  • obu_start_code comment: Fixed 0x01 to 0x000001 to match the 24-bit field width
  • hdr_wcg_idc table (value 2): Replaced confusing "Both HDR and WCG are to be indicated in the stream" with "The video stream is both HDR and WCG"
  • HDR definition: Replaced vague "EOTF higher than BT-1886" with reference to BT-2100 HDR transfer functions (PQ or HLG)
  • Bibliography: Added BT-2100 entry and updated all BT references to use formal ITU-R citation format with version and date

Still to be discussed / done

  • Simplify hdr_wcg_idc table entries 0 and 1: Remove "container" language per Maryla's suggestion (e.g. 0 = "A video that is neither HDR nor WCG, i.e. an SDR video", 1 = "A video that is WCG but not HDR")
  • format_identifier hex notation: Optionally add hex value (0x41563031) alongside the ASCII 'AV01'. Further thoughts on this?
  • obu_size recommendation text: Currently says "easy conversion to MP4" but applies to MP4 and Annex B, wording could be broadened

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 26, 2026

📄 Specification Preview

The specification has been built for this PR!

🔗 Download

Download Build Artifact

📦 What's included:

  • index.html — Specification with SDL syntax tables
  • index.pdf — PDF version
  • diff.html — Visual diff against base branch

Built from commit d8d3142 • Thu, 26 Mar 2026 19:54:12 GMT

@y-guyon
Copy link
Copy Markdown

y-guyon commented Mar 27, 2026

  • format_identifier hex notation: Optionally add hex value (0x41563031) alongside the ASCII 'AV01'. Further thoughts on this?

I find it more precise and convenient to write both representations. Not a big deal either way.

  • obu_size recommendation text: Currently says "easy conversion to MP4" but applies to MP4 and Annex B, wording could be broadened

I agree. Maybe even explain why exactly the conversion is easy.

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.

2 participants