ComCut - Automatically clean recordings of commercials

[NiteShdw]

ComCut 1.2 build 81 beta [Homepage] [Download .EXE only]

Changes:
+ Added automatic framerate detection (from Comskip log)
* Manual framerate setting is used only if not reported by Comskip

[CajunCatfish]

NiteShdw,

You rock! New to the GBPVR world and my first experience was with Comclean. Couldn't get it quite right. 1st install of Comcut and I've been loving it ever since. Many, Many thanks.

Been toying with the idea of selecting the delete original option cause it works so well but I see some have had problems with that. Any thoughts about a delete only on successful completion logic or delete previous recordings by timeframe option?

[SniperDude]

NoJoy yet on these recordings that have been problematic... also no indication they were tried?

log:
1/16/2007 11:01:46 AM - MPEG2Cut - mpgtx exited with error code 1
1/16/2007 11:01:46 AM - ProcessComCut - Error cutting MPEG2 file.
Output file may already exist or commercials have already been removed.
1/16/2007 11:01:46 AM - MPEG2Cut - mpgtx exited with error code 1
1/16/2007 11:01:46 AM - ProcessComCut - Error cutting MPEG2 file.
Output file may already exist or commercials have already been removed.

Example comskip.txt type file:
FILE PROCESSING COMPLETE
------------------------
0 3975
37531 43830
63904 71711
91660 98103
109875 117094
140807 149357
162829 170335
186060 191612
211999 217304


Note: this file is not from ComSkip, but from ShowAnalyzer but the cuts are right on the money

... where is the framerate option? (actually I found it .. 30 or 25.. but not 29.997 or other custom number?)

I am going to run comskip on the files to determine if its something thats missing... but it works on bout 80% of hte files.. so not sure..

anyhow.. just an update

[NiteShdw]

The "Delete original recording and replace with commercial free version" option will ONLY do that if there were no errors during the entire process. Regardless, you'll never lose both the original and the commercial free version. I've thought about adding a "Delete old recordings" option to the Batch Mode. Look for it in a future version.

Regarding the MPGTX errors... I've never used ShowAnalyzer, so I've never had any log files to work with. Could you attach an actual log file or two? There are a few differences between the files from what you posted.

The framerate option does calculate NTSC as 29.97, I just simplified the label on the option screen. There is no option for a custom rate, though there is no reason that I couldn't add it easily. What other framerates would be required (I suppose 50/60 for HD?)

[SniperDude]

Nite,
sorry I didn't mean the problem was yours to solve =)
the only thing that would be helpful on your end.. is a little more data in the log...
What I posted is your (ComCut) error.log file, and a comskip-type file from ShowAnaylzer.... showing that its extremely close to the same type of file as ComSkip generates... the only difference that I can see is the top line is missing #frames and rate... I tried running comskip on these files.. and got close but not close enough to the same results... SA does not generate any log files by default(probably a hidden option somewhere) anyhow...

So in short, I was hoping your log files could be more descriptive, IE: file1.mpg failed because file1_original.mpg was already found. or MPTGX process failed on file "file1.mpg" error code 12. (even perhaps the commandline it was trying process) etc... etc..

then I can just parse the logs after going thru a directory of 20-40 files or more.. and see what videos did not process, and possible causes...

Does that make sense?
Thanks in advance.

PSS: Another slick thing in the batch interface would be to show the list as you do, and put a nice GreenCheck or Red(X) next to the file names that failed... and perhaps have the ability to add our own custom routine or 2 to the whole batch process.

Ok... that should keep ya busy :p

[NiteShdw]

The error that you had is definately related to the MPGTX command line. It's probably calculating the breaks wrong (it has to convert the frame numbers to time intervals). If you wish, I'll release a build that spits out the command line of the executed application when there is an error, so you can retry it yourself.

I've tried to capture the standardoutput within ComCut, but I haven't had a lot of luck with it...

I threw together current Batch Mode in about an hour. I'm definately open for suggestions on improving it (such as the check marks).

[SniperDude]

that would be great... will be monitoring over next few hours =)

[NiteShdw]

Build 82 [Download .EXE]
* Adds custom frame rate option
* Reports command line of external programs in error.log

[SniperDude]

downloaded tested...

two things..

this directory has 3 files in it..
search in batch mode finds all 3.

Batch mode runs on 2 of the 3 ... 1st one ... 3rd one..
Errorlog does show the command line that failed.. (missing 1 space btwn .exe and -j)

Thanks much...
Now I can see the error that is generated by manually running that command line...

Error log reads like this.

Code:

1/16/2007 3:59:37 PM - MPEG2Cut - mpgtx exited with error code 1
Command Line: H:\comcut\mpgtx.exe-j -f -o "H:\TV\The Lost Room\The Lost Room - 1of3- The Key and the Clock.comskip.mpg" "H:\TV\The Lost Room\The Lost Room - 1of3- The Key and the Clock.mpg" [0:2:12-0:20:52] [0:24:22-0:35:32] [0:39:52-0:50:58] [0:54:33-1:1:6] [1:5:7-1:18:18] [1:23:3-1:30:33] [1:34:43-1:43:28] [1:46:33-1:57:53]
1/16/2007 3:59:37 PM - ProcessComCut - Error cutting MPEG2 file.
Output file may already exist or commercials have already been removed.
1/16/2007 3:59:38 PM - MPEG2Cut - mpgtx exited with error code 1
Command Line: H:\comcut\mpgtx.exe-j -f -o "H:\TV\The Lost Room\The Lost Room - 3of3- The Eye and the Prime Object.comskip.mpg" "H:\TV\The Lost Room\The Lost Room - 3of3- The Eye and the Prime Object.mpg" [-0:19:40] [0:23:11-0:36:17] [0:41:2-0:52:51] [0:57:41-1:7:30] [1:11:40-1:18:44] [1:23:59-1:30:20] [1:35:15-1:44:10] [1:48:50-1:58:23]
1/16/2007 3:59:38 PM - ProcessComCut - Error cutting MPEG2 file.
Output file may already exist or commercials have already been removed.

you can see it missed 2of3 - fluke?

you mentioned you might not be able to capture error messages, have you tried just piping them into the error.log? Just Curious



eg:
Command Line: H:\comcut\mpgtx.exe-j -f -o "H:\TV\The Lost Room\The Lost Room - 3of3- The Eye and the Prime Object.comskip.mpg" "H:\TV\The Lost Room\The Lost Room - 3of3- The Eye and the Prime Object.mpg" [-0:19:40] [0:23:11-0:36:17] [0:41:2-0:52:51] [0:57:41-1:7:30] [1:11:40-1:18:44] [1:23:59-1:30:20] [1:35:15-1:44:10] [1:48:50-1:58:23] >> error.log

PSS: Errors were "range [1:48:50-1:58:23] results in invalid [3793759309-4126657332] range"
along that range... havent been able to figure out why yet... but just an fyi
Thanks again!!
-Tim

[NiteShdw]

The missing space is just because I forgot to put it in there when it dumps the command line to the log, it doesn't affect how the program runs.

ComCut will completely skip processing a file for a few reasons:
1. You turned off the "Run Comskip" option, and there was a problem with the log file
2. The file's already been processed at the log file was replaced with a "placeholder" file
3. The output file (.comskip.mpg) already exists

What happens when you run comcut on just the one file that it skipped?

The mpgtx.exe error "range [1:48:50-1:58:23] results in invalid [3793759309-4126657332] range" indicates that the file is shorter than 1 hour, 58 minutes, 23 seconds. The ShowAnalyzer log file doesn't tell ComCut the total number of frames in the file (like Comskip does), so it doesn't when to tell mpgtx. To test this, try using this command line:

Code:

H:\comcut\mpgtx.exe-j -f -o "H:\TV\The Lost Room\The Lost Room - 3of3- The Eye and the Prime Object.comskip.mpg" "H:\TV\The Lost Room\The Lost Room - 3of3- The Eye and the Prime Object.mpg" [-0:19:40] [0:23:11-0:36:17] [0:41:2-0:52:51] [0:57:41-1:7:30] [1:11:40-1:18:44] [1:23:59-1:30:20] [1:35:15-1:44:10] [1:48:50-]

The only change is that the last segment tells mpgtx to include everything from time index 1:48:50 to the end of the file, instead of from 1:48:50 to 1:58:23.