Skip to content
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

Require Period@duration *and* MPD@mediaPresentationDuration if fixed-duration MPD #219

Closed
sandersaares opened this issue Nov 20, 2018 · 4 comments
Assignees
Labels
discussion needed Issue needs general input from IOP members to move forward

Comments

@sandersaares
Copy link
Member

sandersaares commented Nov 20, 2018

IOP currently says that either Period@duration of the last period or MPD@mediaPresentationDuration must be present. The implication being that the client looks up whichever is present and calculates the other accordingly.

This looks to me like needless complexity. All the information to calculate both of these values is going to be present in the MPD. There is no need here for client knowledge or involvement. Let the packager do the calculations so that both values are correct.

The service should bear the burden of such calculations, not the client - you create the MPD once but play it in many different players. We can expect better interoperability if we limit the needless complexity that must be re-implemented and re-interpreted in every client.

Thus I propose we require that both values are present if the last period has a fixed duration.

@haudiobe
Copy link
Contributor

haudiobe commented Dec 6, 2018

This is a postroll use case, where an xlink controls the duration of the last Period. It is also relevant for on-demand content where the duration of the xlink Period are determined by the ad decision system. Options are to define the last Period.

@sandersaares
Copy link
Member Author

This is very painful. It basically makes MPD@mediaPresentationDuration unusable in the timing model, just an optional "maybe you have it maybe you don't" extra.

The use cases feel valid to me. I think we just need to adjust our timing model to not depend on this attribute.

@sandersaares
Copy link
Member Author

I will attempt to adjust the timing principles in #210 not to rely on MPD@mediaPresentationDuration.

@sandersaares sandersaares self-assigned this Dec 19, 2018
sandersaares added a commit to sandersaares/DASH-IF-IOP that referenced this issue Dec 26, 2018
We have to calculate total duration by adding up period durations because we do not know xlink period durations on the service side!

This is not strictly happy but it is what it is.

Closes Dash-Industry-Forum#219
Ref #213
@sandersaares
Copy link
Member Author

Adjusted #210 not to use MPD@mediaPresentationDuration.

@technogeek00 technogeek00 added the discussion needed Issue needs general input from IOP members to move forward label Mar 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion needed Issue needs general input from IOP members to move forward
Projects
None yet
Development

No branches or pull requests

3 participants