Comskip .txt files not being moved

Jan 25, 2014 at 7:23 PM
I'm using 2.3.15 beta with newest comskip donator version (slightly modified comskip.ini to not output an EDL file, since the TXT files work perfectly with XBMC).

I'm doing a simple mp4 conversion and comskip detection, but have it set to "Skip Cutting Ads," since it's not always perfect where it cuts, and I prefer to have the ability to skip back and forth. However, when MCEBuddy has renamed and copied the file, the comskip .txt file is not moved as well, it's simply deleted from the temp "working" folder. Am I missing something in the settings in order to rename and move the comskip file, or do I need to output an EDL file for that to work?
Coordinator
Jan 25, 2014 at 7:54 PM
likely since when there is no EDL file mcebuddy assumes no commercials were detected


Jan 26, 2014 at 1:16 AM
When I have enabled the EDL file output, it correctly renames the EDL file and moves it with the MP4 file, but it is not moving the TXT file. I'm going to do more testing with XBMC and EDL files, but thought it strange the TXT files were still getting deleted.
Jan 26, 2014 at 4:53 PM
I've confirmed XBMC will not recognize EDL files output by Comskip and TXT files are not getting moved, even when EDL files are being created.

Was this fix implemented as planned? Can i upload a log file to help?
Coordinator
Jan 26, 2014 at 8:11 PM
It has been fixed. See this thread




Jan 26, 2014 at 9:12 PM
Interesting. Not sure why the .txt files are not being moved along with the .edl files, then. I'm guessing it's different because the other user is doing comskip along with current recordings in MediaPortal, and I'm having mcebuddy do comskip. So if mcebuddy finds the EDL already there, it won't touch other files (txt) named the same, but if mcebuddy does the comskip detection, it will only move the EDL file. Is that correct?
Coordinator
Jan 27, 2014 at 12:39 AM
Let me check the logic and get back to you

Coordinator
Jan 27, 2014 at 4:00 AM
Okay so MCEBUddy was "deleting" the TXT files that came with the external EDL file which was fixed.

It never really copied the TXT files. What is this TXT file?


Jan 27, 2014 at 4:55 AM
The TXT file is the default output from Comskip. If you have no other options selected, it will output the TXT file.

It's formatted like this:

FILE PROCESSING COMPLETE 546168 FRAMES AT 5994

18 1426
39487 55767
91839 108071
143189 159459
192778 196408
203603 207209
243251 257706
303343 313247
339759 356045
394910 411759
469078 485302
538824 546185
Coordinator
Jan 27, 2014 at 4:13 PM
The latest version of XBMC supports EDL files.


However if you still want to keep the TXT file created by comkip you can write a customcommand to copy the txt file to your desired folder.


Marked as answer by rboy1 on 1/29/2014 at 1:19 AM
Jan 27, 2014 at 7:47 PM
Thanks for checking into this!

XBMC does recognize EDL files, but not in the format output by Comskip, only MPlayer EDL's.

Comskip EDL format:
[start timestamp] [end timestamp] [start timestamp] [end timestamp]...[0]
MPlayer:
0 - Cut
1 - Mute
2 - Scene Marker (if start and end times are specified, the end time is used)
3 - Commercial Break

5.3   7.1    0
15    16.7   1
420   822    3
1     255.3  2
720.1        2
I'll look into customcommands and see if I can make it work. Does it support command line actions like copy/move? Again, thanks for your time!
Coordinator
Jan 27, 2014 at 8:35 PM
This is the same format comskip uses but the code is always 0 (cut)

See the documentation and you can search the forum on how to use custom command. It invokes any command or batch or file you want to execute and you can pass variables to it

Feb 2, 2014 at 6:50 PM
I believe I found why XBMC was not recognizing the EDL files - when comskip is run by MCEBuddy and the EDL file is renamed and moved along with the converted video file, the EDL is renamed using all lower caps, while the movie name is appropriately capitalized (i.e., "Gladiator.wtv" vs. "gladiator.edl"). If I manually capitalize the titles, it works perfectly. Also, if I manually run comskip on a converted file, it correctly capitalizes, just not when moved by MCEBuddy.
Coordinator
Feb 3, 2014 at 12:53 AM
wow that's just crazy (for windows) but let me see what's going on
Coordinator
Feb 3, 2014 at 1:38 AM
Okay I've fixed it, it will maintain the exact same case as the converted file as well. You'll have to wait a bit for the next build since I'm admist fixing some other issues with avidemux which may take a few days before the next beta build is out
Feb 3, 2014 at 2:25 AM
Sounds good - thanks for fixing it! I'll just run a batch rename task on all my EDL files for now :)
Feb 5, 2014 at 5:16 AM
Thanks for the new build - EDL files are being moved and named correctly now!
Sep 12, 2015 at 5:49 AM
Edited Sep 12, 2015 at 6:43 AM
Nevermind, I found a better workaround.
Kodi's support of EDL is slightly broken, so I was going to ask for better .txt support but there's a work-around available by changing some comskip settings.

http://trac.kodi.tv/ticket/15207#no1
Coordinator
Sep 14, 2015 at 1:12 AM
See this for custom command

  • %convertedfile% - will be replaced with the full name and path of the (converted file for PreCustom) (converted and renamed file for Custom) (moved/destination converted file for PostCustom) as part of the parameters passed to the custom program.