twitchtv / acm-mmsys-2020-grand-challenge

Assets for Twitch's ACM MMSys 2020 Grand Challenge
Apache License 2.0
42 stars 20 forks source link

Using latest version of dash.js #10

Closed kefanchen closed 3 years ago

kefanchen commented 3 years ago

Hi, @johnBartos , thank you for setting up an environment to study the low latency streaming.

Can I just replace the dash.js directory in this repo with the latest version to test the new functions of low latency dash? As you know, dash v3.2 has adoptted some great solutions of this challenge, and we want to test them with certain networks. You mentioned that the dash.js in this repo is modified, so I wound like to know is it okay to simply replace the dash.js directory, or some coordination is needed to incorporate the latest dash.js with the rest of this repo?

Thank you for your attention.

johnBartos commented 3 years ago

Hey @kefanchen, you can replace it but you'll need to modify Dash.js to pre-request the segment before it's available. You might be able to skip this but I'm not sure what the impact will be. To be honest I can't remember where I made that change, but I don't remember it being too difficult.

kefanchen commented 3 years ago

Thank you, it works.