Opened 11 years ago

Closed 11 years ago

Last modified 10 years ago

#5809 closed enhancement (wontfix)

Add AVIDemux Project export capability to mythcommflag

Reported by: robert.mcnamara@… Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: mythtv Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

This is a patch to add two new command line options to mythcommflag. They are:

--getskiplist-avidemux

and

--getcutlist-avidemux

These options, combined with --outputfile somefile, will export the cut and skip lists as AVIDemux compatible project files. This is useful for a number of reasons, principally for the lossless commercial cutting of formats like H.264 and XVid produced by several new hardware encoders, most notably the Hauppauge HD-PVR. This simplifies the workflow by generating a working, correct AVIDemux project file, requiring only a "File->Save" in AVIDemux to produce an edited copy of the recording. It can also be accomplished in a user job with a single command line.

Known defects include:

The last frame of the project is always set to frame 1,000,000 since, try as I might, I was unable to get nvp->GetTotalFrameCount?() to return any value. If someone knows the NuppelVideoPlayer? options better than I do, it is very likely a simple fix. To work around the problem I just set the last frame number to something obscenely high. AVIDemux just amends that number to the actual last frame when it opens the project.

Attachments (2)

mythcommflag-avidemux.diff (6.8 KB) - added by robert.mcnamara@… 11 years ago.
Patch adds MythCommflag? AVIDemux export
mythcommflag-avidemux.2.diff (6.8 KB) - added by robert.mcnamara@… 11 years ago.
Slight Improvement (AVI is a better container choice for HD-PVR)

Download all attachments as: .zip

Change History (5)

Changed 11 years ago by robert.mcnamara@…

Attachment: mythcommflag-avidemux.diff added

Patch adds MythCommflag? AVIDemux export

Changed 11 years ago by robert.mcnamara@…

Slight Improvement (AVI is a better container choice for HD-PVR)

comment:1 Changed 11 years ago by cpinkham

Resolution: wontfix
Status: newclosed

Closing ticket after IRC discussion with Robert. A better solution is being worked on and it would probably be best if things like this were in a contrib script rather than hardcoded in the source. Perhaps even a version that took an optional format string to describe the output format of the information rather than hardcoding every desired format.

comment:2 Changed 10 years ago by databubble

I applied mythcommflag-avidemux.2.diff to trunk 20741. Seemed to apply fine, but compile fails:

main.cpp: In function ‘int GetAviDemux?(QString, QString, QString, const QString&)’: main.cpp:317: error: no matching function for call to ‘NuppelVideoPlayer::NuppelVideoPlayer?(const char [16], ProgramInfo?*&)’ ../../libs/libmythtv/NuppelVideoPlayer.h:109: note: candidates are: NuppelVideoPlayer::NuppelVideoPlayer?() ../../libs/libmythtv/NuppelVideoPlayer.h:105: note: NuppelVideoPlayer::NuppelVideoPlayer?(const NuppelVideoPlayer?&)

main.cpp:355: warning: suggest parentheses around ‘&&’ within ‘

main.cpp: In function ‘int main(int, char)’: main.cpp:1382: warning: ignoring return value of ‘int nice(int)’, declared with attribute warn_unused_result main.cpp:1403: warning: ignoring return value of ‘int nice(int)’, declared with attribute warn_unused_result make: * [main.o] Error 1

Any hints?

comment:3 Changed 10 years ago by robert.mcnamara@…

This ticket is closed on my request, as there are better, cleaner ways of handling this. The patch is not being kept current against trunk any more.

Note: See TracTickets for help on using tickets.