-
Notifications
You must be signed in to change notification settings - Fork 7
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
Request for clarification: AdaptationSet@contentType #74
Comments
A tool provided for batch testing of MPDs for specific attributes: https://github.com/Dash-Industry-Forum/Conformance-Software/tree/master/Tools/MPDcheck Conclusion on our test vectors: a significant portion does not have AdaptationSet@contentType, so this attribute should not be mandated to ensure continued compatibility of IOP guidelines to test vectors. The issue should be closed by agreement. |
Proposed edit in Section 3.2.1 |
I think AdaptationSet@contentType should be a mandated attribute. The alternative, "the media type shall be deduced from the @codecs attribute." is not practical. Is every DASH player expected to have a lookup table of possible audio and video codecs? What about when new codecs are introduced - are players in the field (and implemented in hardware in televisions etc) going to get updated retroactively? Since the encoder that is preparing the manifest knows unambiguously what the contentType of each adaptionSet is, it should be required to add this attribute during manifest creation. If this invalidates a significant portion of our test vectors, then we should update our test vectors. |
I generally agree, but what I don't understand is that how is it currently working? A lot of content I have seen does not have this attribute, and players like dash.js still are able to correctly work with such content!? |
Those players must examine the @contentType attribute of the underlying Representations. Is it true that for all vectors where contentType is missing in the AdaptionSet it is present in the Representation elements? |
@wilaw the issue is a bit more mind boggling, please consider this MPD: http://dash.edgesuite.net/dash264/TestCasesHD/2b/DTV/1/live.mpd it has no contentType attribute on any of the levels, while when I test it on dash.js v1.5 or 1.6 it still works perfectly (adaptation set types and representations are all identified.) |
So my assumption is that players like dash.js are then using the representation mimeType and/or codec to figure this out, |
Notes on using @MimeType: a. Is mandatory Notes on conformance testing of @MimeType:
|
AdaptationSet@contentType is an optional DASH attribute. A lot of IOP aspects (e.g. for video and audio components) rely on the presence of this attribute, but its not mandated. Should this be mandated for DASH-IF interoperable content?
The text was updated successfully, but these errors were encountered: