Dash-Industry-Forum / dash.js

A reference client implementation for the playback of MPEG DASH via Javascript and compliant browsers.
http://reference.dashif.org/dash.js/nightly/samples/dash-if-reference-player/index.html
Other
5.1k stars 1.67k forks source link

EBU IRT Keepixo reference live reference streams not playing under 2.0 #1088

Closed wilaw closed 8 years ago

wilaw commented 8 years ago

These streams work under 1.6.0 so this is a regression. Also associated is the failure of EBU-TT-D subtitles under FF covered by #1040 .

http://irtdashreference-i.akamaihd.net/dash/live/901161/bfs/manifestBR.mpd

http://irtdashreference-i.akamaihd.net/dash/live/901161/bfs/manifestARD.mpd

http://212.99.93.126:8090/dash_irt/manifest.mpd

Player loads segments under FF and Chrome, but playback does not begin. No errors are reported in the console logs.

dsparacio commented 8 years ago

Does not play in 1.6.0 for in chrome at all. Not sure this is a regression and may need to be moved to 2.1. More investigation needed at this time.

wilaw commented 8 years ago

Looks like the 1.6.0 claim was a mistake. However, streams do play in 1.4.0, 1.5.0 and 1.5.1 under Chrome.

http://dashif.org/reference/players/javascript/v1.5.1/samples/dash-if-reference-player/index.html?mpd=http://irtdashreference-i.akamaihd.net/dash/live/901161/bfs/manifestBR.mpd

dsparacio commented 8 years ago

Solved issue for stream ARD.mpd, this is the only stream that would playback in any Dash player and did not play in Bitdash or Shaka.

Prior code, preivous to recent change did not set the default bitrate in the dict but now it does bitrateDict[type] = (type === 'video') ? DEFAULT_VIDEO_BITRATE : DEFAULT_AUDIO_BITRATE;

For one this breaks the local storage logic so that will need to be fixed.

The prior code would set 3000 for video and undefined for audio. This is why it breaks….. this stream only has audio at 128 and now we are setting to default 100.

  <Representation id="6" bandwidth="128 000" mimeType="audio/mp4" startWithSAP="1" codecs="mp4a.40.2" audioSamplingRate="48000">
    <AudioChannelConfiguration schemeIdUri="urn:mpeg:dash:23003:3:audio_channel_configuration:2011" value="2" />
  </Representation>
</AdaptationSet>

So the real issue is we need make sure we do not range out of the available bitrates. I am issuing a fix for all of this now.

dsparacio commented 8 years ago

Update, we have logic to not range out is default value is lower then available, the real problem was we are setting invalid value for type fragmentedText. Resolved and fixed the local storage failure as well and made logic cleaner in DOMStorage.

dsparacio commented 8 years ago

For stream http://irtdashreference-i.akamaihd.net/dash/live/901161/bfs/manifestARD.mpd Fixed in PR #1116

WontFix for these two streams until we make sure this is not issue with content. Does not play in any HTML5 Dash player available on the market today. http://irtdashreference-i.akamaihd.net/dash/live/901161/bfs/manifestBR.mpd http://212.99.93.126:8090/dash_irt/manifest.mpd