-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Interstitials improvements and fixes #7425
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
Conversation
7249e99
to
0521fb5
Compare
Tested the PR but playback is (still) stalling for me when total asset length is shorter than playout.
|
- Fix DateRange mapping in Live with fragment reference sliding (use `frag.setStart` over `frag.start =` to update tagAnchor ref) - Skip gaps between interstitial assets - Make MEDIA_ATTACHED async on transferMedia (this.media undefined in id3-track-controller onMediaAttached otherwise) - Add metadata cue timing to assets for better event timing - Removed unnecessary conditionals in changed files (added looser typing in cases where object[index] result may be undefined) Fixes #7426
…s (with no duration)
9c95a57
to
e7621d7
Compare
That should be addressed with the most recent commits. Keep in mind that you will still get stall events your program produces unplayable regions, meaning you should expect a stall error event right before a seek forward to the next interstitial asset after a gap, or to the primary resumption offset. If you receive stall errors and they are not resolved by a seek, let me know. |
…le playing primary
Looks like this is working.
This is still stalling playback for me. Single failing asset is enough to stall playback. |
This PR will...
frag.setStart
overfrag.start =
to updateDateRange.tagAnchor
ref)this.media
undefined in id3-track-controlleronMediaAttached
otherwise)Why is this Pull Request needed?
Fixes Interstitial timing issues in Live. Improves handling of HLS Interstitials with asset lists with inaccurate durations.
Are there any points in the code the reviewer needs to double check?
Resolves issues:
Fixes #7426
Checklist