search

AOMediaCodec / av1-isobmff

Official specification of the AOM group for the carriage of AV1 in ISOBMFF
https://AOMediaCodec.github.io/av1-isobmff
64 stars 16 forks source link

Add a NOTE to clarify the 'mdcv' box #156

Closed wantehchang closed 1 year ago

wantehchang commented 1 year ago

In "their values SHALL match", change the word "match" to "equal (except for different binary representations and precision issues, see the second note below)".

Exchange METADATA_TYPE_HDR_CLL and METADATA_TYPE_HDR_MDCV so that they correspond to mdcv and clli in the right order.

Fix https://github.com/AOMediaCodec/av1-isobmff/issues/138.

Preview | Diff

cconcolato commented 1 year ago

For sample entries corresponding to HDR content, the MasteringDisplayColourVolumeBox mdcv and ContentLightLevelBox clli SHOULD be present , and their values SHALL match equal (except for different binary representations and precision issues, see the second note below) the values contained in the Metadata OBUs of type METADATA_TYPE_HDR_CLL METADATA_TYPE_HDR_MDCV and METADATA_TYPE_HDR_MDCV, METADATA_TYPE_HDR_CLL, if present (in the configOBUs or in the samples) .

change it to:

For sample entries corresponding to HDR content, the following applies:

  • the ContentLightLevelBox clli SHOULD be present. Its values SHALL equal the values contained in the Metadata OBUs of type METADATA_TYPE_HDR_CLL, if present (in the configOBUs or in the samples) .
  • the MasteringDisplayColourVolumeBox mdcv SHOULD be present. Its decoded values SHALL equal (except for precision issues, see the second note below) the decoded values contained in the Metadata OBUs of type METADATA_TYPE_HDR_MDCV, METADATA_TYPE_HDR_CLL, if present (in the configOBUs or in the samples) .
wantehchang commented 1 year ago

Cyril: I made the changes you suggested. Please review.

I made minor edits to your suggestion, on capitalization, fixing an error, and removing extra spaces at the end of sentences.