Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enforce the MIAF-mandated pixel aspect ratio of 1:1 #2484

Open
wantehchang opened this issue Oct 22, 2024 · 3 comments
Open

Enforce the MIAF-mandated pixel aspect ratio of 1:1 #2484

wantehchang opened this issue Oct 22, 2024 · 3 comments

Comments

@wantehchang
Copy link
Collaborator

@leo-barnes @y-guyon

MIAF ISO/IEC 23000-22:2019 says:

7.3.6.5 Pixel aspect ratio

This property shall be supported by the MIAF renderer and may be associated with any image item.

The default pixel aspect ratio of an image, in the absence of association with a PixelAspectRatioBox
property, is 1:1 (i.e. "square"). This value of pixel aspect ratio is the mandatory value for all images
mandated or conditionally mandated by this document, i.e. all images conformant with this document
shall have a pixel aspect ratio of 1:1.

libavif should enforce this requirement. Also, when encoding an AVIF image, libavif can omit the PixelAspectRatioBox property to save space.

@wantehchang
Copy link
Collaborator Author

HEIF ISO/IEC 23008-12:2022 says:

6.5.4.1 Definition

Box type: 'pasp'
Property type: Descriptive item property
Container: ItemPropertyContainerBox
Mandatory (per item): No
Quantity (per item): At most one

When no pixel aspect ratio 'pasp' descriptive item property is associated with an image item, readers
shall infer pixel aspect ratio 1:1 to apply for the image item.

@y-guyon
Copy link
Collaborator

y-guyon commented Oct 23, 2024

when encoding an AVIF image, libavif can omit the PixelAspectRatioBox property to save space

By default it is not written.
You mean rejecting any other value than 1:1 for avifPixelAspectRatioBox when AVIF_TRANSFORM_PASP is set by the user, and never writing pasp?

@wantehchang
Copy link
Collaborator Author

Yes, that's what I meant.

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

No branches or pull requests

2 participants