MPEGGroup / DASHSchema

The XML schema and example XML files for DASH (ISO/IEC 23009-1)
Other
11 stars 16 forks source link

cardinality of PatchLocation@ttl in DASH 5th edition #105

Closed paulhiggs closed 3 years ago

paulhiggs commented 3 years ago

The semantic description of the PatchLocation@ttl attribute is defined in clause 5.15.2 table 49 as

Element or Attribute Name Use Description
@ttl O specifies the time period duration (in seconds) starting from MPD@publishTime until the MPD patch document is at least available at the indicated location above. For details refer to the processing model in subclause 5.14.4.
If not present, the value is unknown.

however it is schematically defined as

<xs:attribute name="ttl" type="xs:double" default="0" use="required"/>

thus, there is no possibility for the ttl attribute to be not present

paulhiggs commented 3 years ago

@technogeek00

mikedo commented 3 years ago

The text is clear. @default is ignored anyway, but should be removed for accuracy.

technogeek00 commented 3 years ago

concur with @mikedo

paulhiggs commented 3 years ago

default="0" is removed from the schema - just needs updating in the FDIS

mikedo commented 3 years ago

The electronic schema still conflicts with the spec text which is clear that it is optional.

technogeek00 commented 3 years ago

I can propose a PR change later tonight

mikedo commented 3 years ago

OK. Don't merge it yet just in case the intent turns out to be the way it is.

technogeek00 commented 3 years ago

@mikedo available as #114

haudiobe commented 3 years ago

FDIS update: removed "default = 0" use = "required"

paulhiggs commented 3 years ago

Schema updated