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.

Sign up for GitHub

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

Jump to bottom

The alpha image item should also have a mandatory ispe property #3

Closed
wantehchang opened this issue Sep 8, 2021 · 0 comments · Fixed by #4
Closed

The alpha image item should also have a mandatory ispe property #3

wantehchang opened this issue Sep 8, 2021 · 0 comments · Fixed by #4

Comments

@wantehchang
Copy link
Contributor

This issue was first reported in Chromium issue 1246678.

If an AVIF image encoded by cavif has an alpha plane, the alpha image item in the AVIF image does not have an ispe property. This is not standards-compliant.

HEIF (ISO/IEC 23008-12:2017) Section 6.5.3.1 says the ispe property is mandatory (per item). See page 11.

MIAF (ISO/IEC 23000-22) Section 7.3.6.3 (Image spatial extents) reiterates that:

ISO/IEC 23008-12 requires that every image item be associated with a property of this type.

See page 15.
wantehchang added a commit to wantehchang/avif-serialize that referenced this issue Sep 8, 2021
@wantehchang
Add a mandatory ispe property to the alpha image
451ff35 
This requires increasing the capacity of the prop_ids ArrayVec in struct
IpmaEntry from 3 to 4.

Fix kornelski#3
@wantehchang wantehchang mentioned this issue Sep 8, 2021
Merged
@wantehchang wantehchang mentioned this issue Sep 12, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add a mandatory ispe property to the alpha image wantehchang/avif-serialize
1 participant
@wantehchang