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

ImaAdsLoader doesn't work with HLS content that uses EXT-X-PROGRAM-DATE-TIME #3865

Closed
priscila225 opened this issue Feb 19, 2018 · 6 comments
Assignees
Labels

Comments

@priscila225
Copy link

priscila225 commented Feb 19, 2018

I'm running the demo app for v2.6.1 and I hardcoded it to play this HLS video: https://content.uplynk.com/babd2bd748ee47c48e0fb6b492245eb9.m3u8

I'm trying to use this google sample vmap: "VMAP - Pre-roll Single Ad, Mid-roll Standard Pod with 3 ads, Post-roll Single Ad"

When the playback starts I see that the prerolls doesn't play, instead I get the 3 midrolls where the preroll should be. Also I can't see the post-roll ad working, it just keeps loading forever.

@andrewlewis andrewlewis self-assigned this Feb 21, 2018
@andrewlewis
Copy link
Collaborator

The content media source has a window that is offset from zero, while the ad group times start from zero. This is not handled correctly in Timeline at the moment, which leads to the player identifying the wrong ad group to play.

We have a fix in review which will treat ad group times as relative to the window in Timeline (ImaAdsLoader already makes this assumption), so hopefully this will be fixed on the development branch soon.

@ojw28 ojw28 changed the title Exoplayer ImaAdsLoader ImaAdsLoader doesn't work with HLS content that uses EXT-X-PROGRAM-DATE-TIME Feb 28, 2018
@ojw28
Copy link
Contributor

ojw28 commented Feb 28, 2018

I think the new plan is to adjust HlsMediaSource so that the Timeline window and period are aligned for this type of media.

@priscila225
Copy link
Author

@andrewlewis @ojw28 sounds good, let me know when you have a fix on HlsMediaSource to test it. Thank you!

andrewlewis added a commit that referenced this issue Mar 2, 2018
Before this change, HlsMediaSource timelines had a period starting at the epoch.
For VOD streams the window position in the period was the program date time.

This change makes period and initial window positions match. For live streams
the window position advances as segments are removed, so its position in the
period is the difference between the initial program date time and the program
date time of the latest playlist.

This also makes it possible to insert ads in VOD HLS content with program date
time, as the period and window are now aligned.

Issue: #3865

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=187590948
@andrewlewis
Copy link
Collaborator

@priscila225 This should be fixed on the development branch. Please try it out and let us know if there are any further issues.

@priscila225
Copy link
Author

priscila225 commented Mar 2, 2018

@andrewlewis this is working fine for us. Thank you a lot! I confirmed using the sample with m3u8 links as well. Also the plan to have it on release, you have a date?

Just to know if we will wait too much to have the fix on final release.
I'll work for now with dev version! awesome

Thank you!

@ojw28
Copy link
Contributor

ojw28 commented Mar 7, 2018

The fix will land in 2.7.1 which will likely be released this week or early next week.

ojw28 pushed a commit that referenced this issue Mar 7, 2018
Before this change, HlsMediaSource timelines had a period starting at the epoch.
For VOD streams the window position in the period was the program date time.

This change makes period and initial window positions match. For live streams
the window position advances as segments are removed, so its position in the
period is the difference between the initial program date time and the program
date time of the latest playlist.

This also makes it possible to insert ads in VOD HLS content with program date
time, as the period and window are now aligned.

Issue: #3865

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=187590948
@google google locked and limited conversation to collaborators Aug 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants