gpac / ComplianceWarden

A pluggable compliance checker (ISOBMFF, HEIF/MIAF/AVIF, AV1 HDR10+)
https://gpac.github.io/ComplianceWarden-wasm/
Other
17 stars 7 forks source link

Rule about `ispe` coming before transform properties not picked up for this image #59

Closed leo-barnes closed 1 year ago

leo-barnes commented 1 year ago

For some reason the rule about ispe coming before transform properties is not picked up for the image mentioned here: https://github.com/strukturag/libheif/issues/784#issuecomment-1477660946 (Direct link https://github.com/bigcat88/pillow_heif/raw/master/tests/images/heif_special/guitar_cw90.hif)

I've definitely seen ComplianceWarden check and warn about this for clap, but from what I can see this image has irot before ispe. Am I missing something, or is this a check that's not implemented yet?

rbouqueau commented 1 year ago

I confirm there is an issue. An early break seems incorrect. Investigating.

rbouqueau commented 1 year ago

Really good catch, thanks for reporting. We even had issues in our own AV1 samples, which are derived from real life samples. That's now fixed.