gpac / ComplianceWarden

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

prepare assert-ids script #77

Closed podborski closed 3 months ago

podborski commented 6 months ago

update script to prepare assert-id's for AV1-ISOBMFF.

podborski commented 6 months ago
$ python aomrules.py --spec av1isobmff --dump
podborski commented 6 months ago

Is there an easy way to combine this output with the existing implementation?

rbouqueau commented 6 months ago

Hi, thank you. Could you resubmit your PR against the master branch? Since av1isobmff (was merged), the current configuration makes it difficult to review. Thanks!

rbouqueau commented 6 months ago

Is there an easy way to combine this output with the existing implementation?

Unfortunately no. We should think about a way to update (that would just insert the assert-ids when done for the first time). Any idea?

podborski commented 6 months ago

Is there an easy way to combine this output with the existing implementation?

Unfortunately no. We should think about a way to update (that would just insert the assert-ids when done for the first time). Any idea?

Not sure, one way would be to see if we can automatically map somehow. I just did change a couple manually after running: python aomrules.py --spec av1isobmff -i https://aomediacodec.github.io/av1-isobmff/v1.2.0.html --dump

podborski commented 6 months ago

Unfortunately I don't have time to finish this mapping myself at the moment. I will be available for any related work next month. If there is anyone who can sit down and map the implemented rules to the existing assert-ids from v1.2.0 this would be cool. If not, I'll try to do it in February.

cconcolato commented 5 months ago

FYI, the build process of the spec now generates and publishes the conformance report using ComplianceWarden, see https://aomediacodec.github.io/av1-isobmff/conformance/

podborski commented 3 months ago

We now have AV1-ISOBMFF version 1.3.0 the we should use instead.