Comskip Transcode NZ Freeview Help required

[Waynesm]

As

[JavaWiz]

Not quite understanding exactly what the question is, or what you are trying to do, however, you have problems in the batch file with the way it's coded.

The tag :eof has special meaning in a batch file, you should change all references of :eof to something else, maybe :eoj or something.

I think you are saying:

• Channels 1-12 - run comskip. Use specific comskip.ini that's tuned to that specific channel.
  
• Channels 5-12 - re-encode with NPVR Encoder.
  
• All other - run comskip, do not re-encode, use generic comskip.ini file
  

Is that what you are describing?

[Waynesm]

Thank you.
I have replaced :eof with goto :eoj and all seems to be working.
Yes I only record on channels 1 to 4 fast turnaround programs eg news docos and specials.
Use the Plus 1 versions of those channels for other programs.
My main question was would a For next type routine be any better for channel selection and comskip ini assignment be any better? or even worthwhile.
Thanks again
Waynesm

[JavaWiz]

I'm glad the :eof changes helped.

Couple notes:
I don't think a for loop helps in this situation, you are not trying to iterate over a collection of items, you have a single channel you want to perform a specific set of steps on.
This process may work better in PostProcessing.bat, unless you want to leverage the comskip files while watching (semi)-live TV

I've tried to simplify the logic in the following file. I have not tested this, so it may not work exactly right the first time, but it may give you an idea how to streamline the batch file:

Code:

@Echo off
GoTo :BEGIN
=================================================================================================
ParallelProcessing.bat
    Run as soon as a recording begins. Commonly used for adding commercial skipping functionality
    (comskip) or transcoding.
    
-------------------------------------------------------------------------------------------------
Arguments:
1.filename        - the full path and file name of the recording file.
2.channelNumber   - the channel number it was recorded from.
3.oid             - unique identifier to the scheduled_recordings table, applicable even if
                    filename (%1) changes
4.capture Oid     - unique identifier to the device used for recording
5.recording title - the title of the recording

-------------------------------------------------------------------------------------------------
Modification History
Date    By         Description
------ ---------- -------------------------------------------------------------------------------
103015 JavaWiz    Initial Template
071316 Waynesm Added code from various posts including NPVR Encoder by Revenz
=================================================================================================
:BEGIN
    SETLOCAL
    Set SCRIPTNAME=%0
    Set BASEDIR=c:\Users\Public\NPVR
    Set LOGFILE=%BASEDIR%\Logs\BatchProcessing.log
    Set COMSKIP_DIR=c:\Comskip
    
    Set FILENAME=%1
    Set CH_NUM=%2
    Set OID=%3
    Set DEV_OID=%4
    Set REC_TITLE=%5
    
    Call :LOGMSG  Recording started - filename: %FILENAME%  ChannelNUM: %CH_NUM%  OID: %OID%  DeviceOID: %DEV_OID%  Title: %REC_TITLE%
    
    REM If channel logo file exists, create parameter for comskip call
    If EXIST %COMSKIP_DIR%\Channel%CH_NUM%.logo.txt (
        Set CHNL_LOGO_PARM=--logo=Channel%CH_NUM%.logo.txt
    ) Else (
        Set CHNL_LOGO_PARM=
    )

    REM Set comskip.ini file`
    If EXIST %COMSKIP_DIR%\Channel%CH_NUM%.ini (
        Set COMSKIP_INIPARM=--ini=Channel%CH_NUM%.ini
    ) Else (
        Set COMSKIP_INIPARM=
    )

:COMSKIP
    Call :LOGMSG  BEGIN Comskip processing for %FILENAME%
    cd /d %COMSKIP_DIR%
    Comskip %COMSKIP_INIPARM% %CHNL_LOGO_PARM%
    Call :LOGMSG  END   Comskip processing for %FILENAME%
    
        REM Only re-encode channels 5-12
    If %CH_NUM% LEQ 4 GoTo  :ENDJOB
    If %CH_NUM% LEQ 13 GoTo :CHK_HANDBRAKE
    GoTo :ENDJOB
    
:CHK_HANDBRAKE
    REM Loop until no HandBrake processes are running
    tasklist | find /c "HandbrakeCLI" > %OID%.txt
    set /p count= <%OID%.txt
    if %count% geq 1 (
      echo ...%count% HandbrakeCLI running now, waiting 15 secs >>processing.log
      sleep 15
      goto :CHK_HANDBRAKE
    )
    del %OID%.txt

:ENCODE    
    cd /d "C:\Users\Public\NPVR\Scripts
    Call :LOGMSG  BEGIN NextPvrEncoder processing for %FILENAME%
    NextPvrEncoder\NextPvrEncoder %FILENAME%
    Call :LOGMSG  END   NextPvrEncoder processing for %FILENAME%

    GoTo :ENDJOB

--------------------------------------------------------------    
:LOGMSG
    echo %*
    echo %date:~4% %time% [%SCRIPTNAME%] %* >> %LOGFILE%
    Goto :EOF
    
--------------------------------------------------------------    
:ENDJOB
    Call :LOGMSG  Recording Ended - filename: %FILENAME%  
    
    REM taskkill /F /IM Comskip.exe
    Exit

[Waynesm]

Hi,
Many thanks for this.
Will try out ASAP
Cheers
Waynesm