SA, Comclean, RenameRecordings...

[afriend1]

From Jere

Turns out that I already have! Just use the comskip proxy. What everyone her e is trying to do is exactly why I wrote that silly little program in the first place. I just tried it and it worked as advertised.

I reinstalled the proxy. It rewrote my registry. I assume this is what it's supposed to do. But it ran comskip .72 that I had in the gbpvr directory. What did I do wrong?

[Jere_Jones]

I reinstalled the proxy. It rewrote my registry. I assume this is what it's supposed to do. But it ran comskip .72 that I had in the gbpvr directory. What did I do wrong?

When you say that you reinstalled the proxy, what did you do?

The comskip proxy does read the registry so that it can find SA no matter where it is run from, but it doesn't write to the registry. What portion did it rewrite?

Jere

[Jere_Jones]

OK, so I use comskip proxy to call SA, instead of having the showanalyzer line in postprocessing.bat...correct?

Right. That's the plan.

I have ordered an MCE bundle w/ PVR250 from PCAlchemy that should show up Saturday. I'll install the card in one of my spare computers so that I can actually get GB-PVR up and running (since I can't install it on the HTPC or the wife will kill me and it GB-PVR won't run on my development machine). Then I'll should be able to verify the advice I'm giving.

Jere

[reboot]

Comskip proxy does run (calls SA), but comclean does not run, and rename recordings eventually runs, although it cannot seem to rename the actual mpg...
Example:
Show ends.
Comskip proxy opens SA.
SA runs, generates the .txt and .wme (as I have it set up).
Comclean runs, and craps out with errors (see first post).
Rename recordings then renames the .txt, .wme, and .log to the episode name, but does NOT rename the .mpg, it simply renames it to showname_original.mpg
This is exactly what it's supposed to do, AFTER comclean has successfully run on the show.
Comclean just won't work properly, for whatever reason, when I use SA.
It works every time when I use comskip.

Ideally, I would like things this way:
Rename recordings does it's thing on the mpg.
(I would also like it if renamerecordings included the episode number, which seems to be in the data from zap2it)
SA then runs, and generates the .txt and .wme with the correct names.
Comclean then runs, and does it's thing.

Renamerecordings is seriously important to me, otherwise I have no idea what the season episodes are named, nor which order they go in.
SA is ultimately the easiest method of detecting the ads.
Comclean does the editing for me, a serious time saver!

With 4 tuners, I do a LOT of recording, and much of it get's archived to DVD.
I really need to get these 3 programs working together, so any help is muchly appreciated!

[reboot]

@ jsteele
Wouldn't it make more sense to have the timeout shorter?
If I set it to 8 minutes, (about the time it takes per hour of recording), and it doesn't find the "file processing complete", then it only waits another 8 minutes before testing again...

[reboot]

I tried your postprocessing.bat set up this way:

Code:

comskip %1

:test_file
find "FILE PROCESSING COMPLETE" %1
goto result%ERRORLEVEL%
exit

:result1
rem if the line is NOT in the file it will do this stuff
rem timeout.exe is part of the XP Resource Kit - free as I recall
rem you can set NNNN to the number of seconds to wait for SA to finish
rem once it times out it will go back and test the file again

timeout.exe 480

goto test_file

:result0
rem if the line is in the file then it will do the rest of your stuff

call comclean %1 >>comclean.log 2>>comclean.err
RenameRecordings %1 > renamerecordings.log

Seems that SA ran, but nothing else happened.
Do I need to download and install the timer thingy, or is there an error in the code?

[Jere_Jones]

Comclean just won't work properly, for whatever reason, when I use SA.
It works every time when I use comskip.

Wondering about this, I went off in search of the source code for comclean. Little did I know it's only a batch file. Good news. Batch files are easy to analyze. I noticed the part that looks for the line with the dashes. It looks for a line with a certain number of dashes. That may be what is causing the problem. The original "specification" for the comskip txt files was that the cut points immediately follow the last line containing only dashes. Comclean seems to expect a certain number of dashes. A modification of that line to expect the number of dashes that SA puts out or a more robust matching line (like only match the first character) might work.

Jere

[reboot]

AHA! Light at the end of the tunnel...so this line

Code:

if "%1"=="-------------------" goto :eof

should be changed...but how? Count the dashes?

[dirtygreek]

I think that the only problem I'm having (which was similar if not exactly the same as reboot's) was related to the number of dashes (-----) in the .txt file that SA was generating. Good job, Jere, noticing that!

I didn't figure the number of dashes would matter, but turns out it does, because the comclean batch file looks for an exact match

if "%1"=="-------------------" goto :eof

(19 dashes)

But SA is outputting too many dashes:

------------------------

(24 dashes)


My simple fix, for now, was to replace the dashes in the comclean.bat with the dashes in one of my SA .txt file - and it worked! Perfectly!

Jere, my suggestion would be to modify your next version of SA to output 19 dashes.

[reboot]

So let me get this straight.
I edit comclean.bat to include 24 dashes.
Do I then still need the timer.exe delay thing? (BTW, I couldn't find it for XP, but when I ran the postprocessing.bat with it in, something called find.exe chewed up about 50% of CPU cycles for half an hour before I manually stopped it).
Timeout.exe also seems to quit as soon as I hit any key (according to the websites I found), which won't work at all...
Comclean, and renamerecordings never did go...
Testing again now.