flashls icon indicating copy to clipboard operation
flashls copied to clipboard

negative duration computed problem

Open Lirui072037 opened this issue 9 years ago • 12 comments

Hi,

What causes the emergence of such a log?

image thank you!

Lirui

Lirui072037 avatar Sep 09 '15 11:09 Lirui072037

it means that there is a difference between fragments duration exposed the playlist (#EXTINF:...) and the real duration of the fragments (extracted from fragment parsing)

mangui avatar Sep 09 '15 11:09 mangui

Is the cause of the stream?Can I change it in the code?

Lirui072037 avatar Sep 09 '15 11:09 Lirui072037

it is a stream issue, you can't workaround in the code. flashls can handle small drifts, but accumulated drifts should not exceed one fragment duration (this is what is happening in your case)

mangui avatar Sep 09 '15 12:09 mangui

But jwplayer won't appear this problem, is the same stream

Lirui072037 avatar Sep 09 '15 12:09 Lirui072037

up to you to use jwplayer, but this stream seems broken. u can eventually share it for confirmation.

mangui avatar Sep 09 '15 12:09 mangui

this is url: http://hz-vod.cdn.aodianyun.com/m3u8/0x0/kekakik.stream.1441438685/kekakik.stream.1441438685.m3u8

Lirui072037 avatar Sep 09 '15 14:09 Lirui072037

@mangui , is this message accurate? It says there should be some duration drift.

Also, is error level logging appropriate for this? Because if hls.js can handle some drift up to a certain amount, perhaps it can be just a warning?

colejona avatar Oct 12 '16 21:10 colejona

Hi @colejona well, if computed duration becomes negative, that means that on the overall playlist, you got a cumulated drifts larger than a frag duration. it might not lead to any visible issue, but it is definitely not a good sign

mangui avatar Oct 12 '16 21:10 mangui

Thanks, @mangui. We're considering converting these error messages into warning messages in our custom logger -- do you think that if we did encounter a visible issue, we would get a different error from hls.js? I wouldn't want to hide the only error that we get when a visible issue occurs..

colejona avatar Oct 12 '16 21:10 colejona

you can convert to a warning and send a PR.

mangui avatar Oct 12 '16 22:10 mangui

Will do, thanks!

colejona avatar Oct 12 '16 22:10 colejona

@mangui I just realized that this is a flashls issue -- we're working with hls.js right now, so it is there that we will submit a pull request for this.

colejona avatar Oct 12 '16 22:10 colejona