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

master/TBA remaining issues #531

Closed
17 of 19 tasks
brandonocasey opened this issue May 30, 2019 · 14 comments
Closed
17 of 19 tasks

master/TBA remaining issues #531

brandonocasey opened this issue May 30, 2019 · 14 comments
Labels
planned planned to be worked on in the future
Milestone

Comments

@brandonocasey
Copy link
Contributor

brandonocasey commented May 30, 2019

issues

mux.js

improvements

Pushed Back

  • fmp4 and ts with handlePartialData. The issue here is that during handlePartialData with muxed content browsers expect the initSegment that we generate to contain the same tracks as they did previously. For non partial we can set the Coalesce stream to remux, which makes it so that the initSegment has both tracks. For the partial transmuxer it is much harder because we don't have a Coalesce stream.
    • We can add a Coalesce stream and check if we are generating content that should have audio and video and then just mock out a fake track for what we will generate later.
    • We may have to wait for audio data before we can emit any data so that we can emit an initSegment with both tracks.

testing

  • Test ABR with handlePartialData
@brandonocasey brandonocasey added this to the TBA/LHLS milestone May 30, 2019
@ldayananda
Copy link
Contributor

are these two handled by the same PR?

  • Video freezes at the start of playback, but audio starts (advanced fmp4 bipbop)
  • buffers after starting playback, unless you hit play (advanced fmp4 bipbop)

@brandonocasey
Copy link
Contributor Author

yes

@mparisi76
Copy link
Contributor

I thought someone said #425 would be included in this release, but I don't see it in the list above. Any plans to address that issue?

@gkatsev
Copy link
Member

gkatsev commented Jul 3, 2019

We haven't had a change to look at #425, it would've potentially been fixed by this but definitely possible it hasn't. Is that still a problem happening in master?

@stale
Copy link

stale bot commented Sep 24, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the outdated label Sep 24, 2019
@ldayananda ldayananda added planned planned to be worked on in the future and removed outdated labels Sep 24, 2019
@brandonocasey
Copy link
Contributor Author

It seems like #425 is fixed in master. I added a manifest to test the file with http://d2zihajmogu5jn.cloudfront.net/bad-pmt/index.m3u8 although it's possible I did not create an accurate reproduction.

@brandonocasey
Copy link
Contributor Author

#624 (comment) is fixed by videojs/mux.js#317

@danrossi
Copy link

I tested this with wowza but not sure if its requesting the sub fragments properly. how can I test it to confirm it's requesting properly ?. The LHLS urls are working at least with these updates.

@gkatsev
Copy link
Member

gkatsev commented Mar 24, 2020

Unfortunately, we do not have full support for Apple's or the open LHLS specs.

@danrossi
Copy link

So it's not fully working yet apart from requesting fragments ? It's playing but don't know if its requesting properly. What requests are we expecting to see ?

@gkatsev
Copy link
Member

gkatsev commented Mar 24, 2020

The reason this is called LHLS is because it was done in service to getting support for low latency streaming, such as being able to download and use parts of segments. Yes, it's confusing, but it kind of stuck. However, actual support for the specs isn't included yet. That means we don't handle anything related to the parts stuff in Apple's LHLS or the prefetch tags from the open source spec.
Both of those would be treated as just regular HLS streams currently.

@gkatsev gkatsev changed the title TBA/LHLS remaining issues master/TBA remaining issues Mar 24, 2020
@danrossi
Copy link

OK. Well it's requesting LHLS wowza streams fine, but we should expect partial requests then when it's working.

@gkatsev
Copy link
Member

gkatsev commented May 13, 2020

@gkatsev gkatsev modified the milestones: TBA/LHLS, 2.0 May 21, 2020
@gkatsev
Copy link
Member

gkatsev commented Jun 10, 2020

Closing as complete. There's just the two PRs in the 2.0 milestone left, and we can use the milestone to track any other work.

@gkatsev gkatsev closed this as completed Jun 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
planned planned to be worked on in the future
Projects
None yet
Development

No branches or pull requests

5 participants