[FEATURE] ConvertCutListToKeepList: Don't skip small segments when commercial cutting is disabled


(Brian) #1

Hi. The summary of my question is: ComSkip found a commercial segment that runs to the end of the recording, and that last chapter is missing after an MCEBUDDY conversion.

I found the following entries in my log (below).
The first four entries list the 4 entries in the EDL file correctly.
The last entry is then discarded as “Skip end segment, too small”

I took this to mean that the end of the commercial cut is later than the end of the video (it is, by about 2 seconds from looking at it in a viewer).

This is certainly livable, but I’m wondering why another chapter would not be created that just ends 1 second before the end of the entire show? It would be nice for this chapter to not be ignored, don’t know if that is possible with any existing options or not.

thanks
BrianGGG

2018-02-10T17:29:12 MCEBuddy.CommercialScan.EDL --> ParseEDL: Cut Segment Start:177.010 End:325.860
2018-02-10T17:29:12 MCEBuddy.CommercialScan.EDL --> ParseEDL: Cut Segment Start:614.750 End:794.330
2018-02-10T17:29:12 MCEBuddy.CommercialScan.EDL --> ParseEDL: Cut Segment Start:1047.710 End:1204.020
2018-02-10T17:29:12 MCEBuddy.CommercialScan.EDL --> ParseEDL: Cut Segment Start:1455.950 End:1548.450
–> ConvertCutListToKeepList: Keep segment Start:0.000 Stop:177.010
–> ConvertCutListToKeepList: Keep segment Start:325.860 Stop:614.750
–> ConvertCutListToKeepList: Keep segment Start:794.330 Stop:1047.710
–> ConvertCutListToKeepList: Keep segment Start:1204.020 Stop:1455.950
–> ConvertCutListToKeepList: Skip end segment, too small Start:1548.450 Stop:1550.068


(Goose) #2

The issue lies with the fact that if MCEBuddy tries to “cut” a segment that too small often the encoder (ffmpeg, handbrake etc) fail due to various reasons. So to avoid this issue we put in a minimum segment size. In your case you aren’t actually cutting but marking the chapters, however while commercial detection the smaller segment are ignore. I guess it’s an optimization that could be done to preserve that information if the user has instructed mcebuddy to just mark the chapters instead of cutting them.

You can open a feature request to mark commercial chapters which are too small to cut


(Goose) #3

This has been fixed in 2.4.9 release. If there are no commercials to be cut, there is no minimum segment size enforced.

Also we’ve added an option to adjust the minimum segment size in the System Settings page.