Dash-Industry-Forum / CMAF-Conformance

CMAF sub repository of DASH-IF-Conformance. This repo contains CTAWAVE conformance functionalities as a submodule. CTA WAVE Github project is located at https://github.com/orgs/Dash-Industry-Forum/projects/6 but it is not visible to the public. To see this page & contribute, please get in contact with repository admin to be added as a collaborator.
Other
9 stars 3 forks source link

Spurious errors reported for valid content where samples don't begin at the start of the mdat #17

Closed sdp198 closed 4 years ago

sdp198 commented 6 years ago

Content for which the media samples are correctly referenced in the TrackRunBox but which don't begin at the very start of the mdat box are incorrectly reported with errors such as:

(10 repetition\s) **'CMAF check violated: Section 7.3.2.3- All media samples in a CMAF Chunk SHALL be addressed by byte offsets in the TrackRunBox relative to first byte of the MovieFragmentBox', but not found for Rep 896x504p25 Chunk 18

In the chunk in question here there are some bytes of private (ie not referenced from a trun) data at the start of the mdat. It appears that the check is based on adding up the size of boxes ahead of the mdat and checking that length is what appears in the baseDataOffset. Instead it might be better to just check that this value leads to a position within the mdat and that the total length of content in the trun doesn't, when added to this value, go beyond the end of the mdat.

sdp198 commented 6 years ago

Also this message appears in the "CMAF Compared representations validation error" report, but in reality it's not related to comparing representations, but to compliance of individual CMAF Tracks in their own right.

eceozturk commented 4 years ago

This issue is addressed as suggested via PR #54.

Closing this issue now. Please reopen if any related problems/questions appear.