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

[DASH] Partials manifest updates not supported, broken playback #1317

Open
1 of 7 tasks
dapharsyde opened this issue Jul 6, 2023 · 3 comments
Open
1 of 7 tasks

[DASH] Partials manifest updates not supported, broken playback #1317

dapharsyde opened this issue Jul 6, 2023 · 3 comments
Labels
Issue Type: Improvement issue has requested to improve an existing functionality

Comments

@dapharsyde
Copy link

Bug report

Describe the bug

Here is a clear and concise description of what the problem is:

Playback of Hulu LiveTV content with IA results in playback jumping backwards when advertisements begin playing.

Expected Behavior

Here is a clear and concise description of what was expected to happen:

Playback should continue smoothly - or just skip the ads/play a blank screen ;)

Actual Behavior

Playback jumps backwards and forwards, eventually the video playback ends. Issue occurs when manifest files begin including presentationTimeOffset.

Possible Fix

Implement support for presentationTimeOffset

To Reproduce

Steps to reproduce the behavior:

  1. Begin Live TV on Hulu channel that marks advertisements
  2. Wait for advertisement to begin playing

Debuglog

The debuglog can be found here:

kodi.log

MPD/M3U8s/ISM

An example or copy of a manifest (or manifests for HLS - master and variants) can be found here:

manifests.zip

Screenshots

Here are some links or screenshots to help explain the problem:

Additional context or screenshots (if appropriate)

Here is some additional context or explanation that might help:

Your Environment

Used Operating system:

  • Android

  • iOS

  • tvOS

  • Linux

  • OSX

  • Windows

  • Windows UWP

  • Operating system version/name: Windows 10

  • Kodi version: 21.0

  • inputstream.adaptive version: Omega nightly build

note: Once the issue is made we require you to update it with new information should that be required.
Team Kodi will consider your problem report however, we will not make any promises the problem will be solved.

@CastagnaIT
Copy link
Collaborator

code removed by PR #564 need more investigation

@dapharsyde
Copy link
Author

Here are some patches that at least get the value of PresentationTimeOffset parsed out of the SegmentTemplate, but I don't know how to incorporate it into the clock timing. I'm hoping it's just a matter of some adding/subtracting some time values but this is all over my head. @matthuisman - can you explain why this was removed in #564 and what it would take to add it back in? Would it break streams from other providers?

DASHTree.cpp.patch
SegTemplate.cpp.patch
SegTemplate.h.patch

@CastagnaIT
Copy link
Collaborator

CastagnaIT commented Mar 28, 2024

I looked again at this problem
i see that this provider send "partials" manifest updates
if you look at subsequent manifests they "lack" of many data like representation tags
this is the cause of following warnings

2023-07-06 01:36:32.228 T:4304  warning <general>: AddOnLog: inputstream.adaptive: adaptive::CDashTree::ParseTagAdaptationSet: Skipped AdaptationSet with id: "4289974598", has no representations.
2023-07-06 01:36:32.228 T:4304  warning <general>: AddOnLog: inputstream.adaptive: adaptive::CDashTree::ParseTagAdaptationSet: Skipped AdaptationSet with id: "2008774996", has no representations.
2023-07-06 01:36:32.228 T:4304  warning <general>: AddOnLog: inputstream.adaptive: adaptive::CDashTree::ParseTagAdaptationSet: Skipped AdaptationSet with id: "901119627", has no representations.
2023-07-06 01:36:32.228 T:4304  warning <general>: AddOnLog: inputstream.adaptive: No periods in the manifest

atm "partials" manifest updates are not supported on ISA
and so must be implemented from scratch

since you have already provided the manifests will be easier for the future do the implementation
and so presentationTimeOffset should not be a problem

this issue is similar to #1426

@CastagnaIT CastagnaIT changed the title [DASH] SegmentTemplate presentationTimeOffset not implemented [DASH] Partials manifest updates not supported, broken playback Mar 28, 2024
@CastagnaIT CastagnaIT added the Issue Type: Improvement issue has requested to improve an existing functionality label Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue Type: Improvement issue has requested to improve an existing functionality
Projects
None yet
Development

No branches or pull requests

2 participants