Page 1 of 3

EDL file skip / comskip broken on >= 3.0

Posted: 29 Jun 2017, 06:38
by morbo1980
Hi,

Ever since the update to version 3 of MrMC I've had a problem that EDL file handling (generated from comskip) doesn't work correctly. This is on Apple TV but I've confirmed that it is also an issue on iOS. The problem still exists on 3.0.7 on Apple TV.

What happens is as follows...

* Video (tv recording) is playing
* MrMC will show on screen that it's about to skip ahead (over a commercial break)
* The skip will appear to happen, but now MrMC will sit as if paused on the first frame after the skip
* If you don't do anything eventually it will resume playing from the point after the skip (after the time has elapsed that was skipped)
* The wait can be bypassed by doing a big skip twice (ie: hit up on the remote twice). The video will resume playing from the end of the skip.

This issue was initially in Kodi 17 and fixed in 18 dev and then backported to 17.1 according to several posts on the Kodi forums. The specific pull requests that are supposed to implement the fix are as follows...
https://github.com/xbmc/xbmc/pull/11646
https://github.com/xbmc/xbmc/pull/11714

Would it be possible to port this fix into MrMC?

Thanks,
Ben

Re: EDL file skip / comskip broken on >= 3.0

Posted: 29 Jun 2017, 07:43
by amet
Hi,

we will look into it, it might not be as simple as using Kodi commits as our internal player engine is a bit different then theirs.... I'll let @davilla answer this, but no reason we can't look into fixing it.

Are you playing this file via PVR backend? does the same issue happen if you play it from a folder? if so, could you share the recording and EDL file with us? contact us on [email protected] if you can, then we can reproduce and fix.

thanks for letting us know there is an issue :)

amet

Re: EDL file skip / comskip broken on >= 3.0

Posted: 29 Jun 2017, 13:21
by davilla
These can be back-ported. I was waiting to see if anyone complained as I don't use EDL. Best get into testflight where you can test some blind commits.

Re: EDL file skip / comskip broken on >= 3.0

Posted: 05 Jul 2017, 17:39
by davilla
grr, those patches are messy and touch too much to do blind.

To fix this, we need the recording and EDL files.

Re: EDL file skip / comskip broken on >= 3.0

Posted: 11 Jul 2017, 08:58
by morbo1980
Sorry I haven't been on earlier to follow up on this. I'm currently (slowly) uploading a video to OneDrive that shows the problem. Here's the link for when it finishes https://1drv.ms/f/s!Ar6IzANKIbrXjlnOs09uV4MXk1N0

Playing the file from the beginning will do a 2:15 jump that works correctly to start the video from after the initial ad break. The next ad break is soon after the 12 minute mark and that auto skip will show the problem.

Re: EDL file skip / comskip broken on >= 3.0

Posted: 11 Jul 2017, 17:02
by davilla
morbo1980 wrote: 11 Jul 2017, 08:58 Sorry I haven't been on earlier to follow up on this. I'm currently (slowly) uploading a video to OneDrive that shows the problem. Here's the link for when it finishes https://1drv.ms/f/s!Ar6IzANKIbrXjlnOs09uV4MXk1N0

Playing the file from the beginning will do a 2:15 jump that works correctly to start the video from after the initial ad break. The next ad break is soon after the 12 minute mark and that auto skip will show the problem.
Let me know here when video is uploaded, thx.

Re: EDL file skip / comskip broken on >= 3.0

Posted: 12 Jul 2017, 00:21
by morbo1980
The video has finished uploading.

Thanks.

Re: EDL file skip / comskip broken on >= 3.0

Posted: 12 Jul 2017, 01:45
by davilla
morbo1980 wrote: 12 Jul 2017, 00:21 The video has finished uploading.

Thanks.
ok, will pull it down when I get back on a fast net tomorrow. Will reply when I have it.

Re: EDL file skip / comskip broken on >= 3.0

Posted: 12 Jul 2017, 20:57
by davilla
Got it, issue reproduced. Possible fixed it but I have to do more extensive testing as this touches critical areas in the player code.

Re: EDL file skip / comskip broken on >= 3.0

Posted: 13 Jul 2017, 13:06
by davilla
3.0.9 (170713.340)
fix 206 HTTP we are getting from RSS feeds
fix possible crash when checking CheckLibLoadedAndLoad()
fix crashes in glrKillClient when we try to render while in background on iOS/tvOS
fix EDL lists
regression - bring up channel list on LiveTV on Select(hold)

fix is in testflight if you want to give it a try.

viewtopic.php?f=10&t=1253