MPEGGroup / FileFormat

MPEG file format discussions
20 stars 0 forks source link

Clean Aperture semantics should be clarified #77

Open cconcolato opened 1 year ago

cconcolato commented 1 year ago

The current semantics indicate:

The pixel aspect ratio and clean aperture of the video may be specified using the PixelAspectRatioBox and CleanApertureBox sample entry boxes, respectively. These are both optional; if present, they over-ride the declarations (if any) in structures specific to the video codec, which structures should be examined if these boxes are absent.

I find the sentence "overriding structures specific to the video codec" unclear, especially for Clean Aperture. I see 2 use cases related to cropping:

I think the ISOBMFF spec should be update to say that:

podborski commented 8 months ago

in-stream cropping features that are optional, e.g. as metadata or sei, should be reflected at the container level (i.e. values should match) and if they don't the container should take precedence.

The container shall always take precedence, no?

cconcolato commented 6 days ago

The group discussed and wonders how real-world implementations would behave if the clap would ask to crop a stream (512 px) with dimensions larger (e.g. 508px) than the cropped dimensions indicated by the in-stream structures (e.g. 500px). We also note that ISOBMFF should not be dictating player behavior, but rather document the stream, as integration of decoder and cropping tool may be different from implementation to implementation. We welcome further inputs on how to improve this text. We may want to create conformance files to test various player behaviors.