-
Notifications
You must be signed in to change notification settings - Fork 1
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
Relax property association order on readers #113
Comments
Hi @y-guyon, I agree that the sentence in 6.5.1, which appeared already in HEIF v1, is problematic. I think it might have tried to say that even though descriptive properties are specified in 6.5 of HEIF v1 to appear before any transformative properties, future versions of HEIF or other specifications may specify a descriptive property propA that follows a transformative image property and describes the output of that transformation, in which case HEIF v1 readers shall not misinterpret the descriptive property propA to describe the reconstructed image (before transformations). The changes you are proposing are going to the right direction IMO. Additionally, I'd propose to add an explanation to the NOTE that you proposed: NOTE: It is uncertain if readers would be able to correctly interpret descriptive properties that follow the first transformative property or the first unrecognized essential property, whichever is earlier, in the sequence associating properties with an item, because those descriptive properties possibly describe the output image after the transformation(s). In previous versions of this document, readers had to allow and ignore descriptive properties following the first transformative or unrecognized property, whichever is earlier, in the sequence associating properties with an item. |
We agree to include this text into the Amd2 of HEIF. |
Section 6.5.1 of HEIF says:
This leads to some issues:
essential
property which is an "item property that readers are required to process", and "readers shall allow and ignore [potentially these same essential] descriptive properties".Saying that descriptive properties describe items before any transformation should be enough in my opinion.
HEIF amendment suggestion
The text was updated successfully, but these errors were encountered: