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

Confusing text on RenderWidth and RenderHeight #120

Closed cconcolato closed 2 years ago

cconcolato commented 4 years ago

The current spec says:

The width and height in the TrackHeaderBox SHOULD equal, respectively, the maximum RenderWidth, called MaxRenderWidth, and the maximum RenderHeight, called MaxRenderHeight, of all the frames associated with this sample entry. Additionally, if MaxRenderWidth and MaxRenderHeight values do not equal respectively the max_frame_width_minus_1 + 1 and max_frame_height_minus_1 + 1 values of the Sequence Header OBU, a PixelAspectRatioBox box SHALL be present in the sample entry and set such that

This is not correct as there can be multiple sample description entries for a given TrackHeaderBox. It should probably say:

of all the frames in the track.

cconcolato commented 2 years ago

The group agrees with the proposed change.