Version 4 patch attached; this should be final (!).
Small diff relative to v3; supports per-recording-profile,
per-schedule transcoding. This supports the notion of "auto-transcode
this show if it was recorded on an HDTV tuner, but don't transcode if
it was recorded on an analog card". This ended up actually cleaning up
some code in the previous diff.
Thanks,
--Rob
On Sun, May 01, 2005 at 06:50:14PM -0400, Robert Tsai wrote:
> Version 3 patch attached.
>
> Small diff relative to v2; fixed a corner-case crash involving live
> TV and a freshly-installed database with no codecparams for
> recording profiles.
>
> On Sat, Apr 30, 2005 at 03:04:39PM -0400, Robert Tsai wrote:
> > Version 2 patch attached.
> >
> > Also:
> >
> > - Change JobQueue status to sort by inserttime rather than
> > starttime. Jobs may be manually queued in an order other than
> > when the recordings were broadcast, and I think the "Backend
> > Status" page would make more sense to show the job queue
> > sorted by what has just happened and what is about to happen.
> >
> > Short-comings of the current system that have not been addressed
> > by this patch (e.g., should be no change in behavior or
> > functionality):
> >
> > - No way to only transcode a show if it was recorded by a
> > certain device. For example, one might want to only transcode
> > HDTV recordings, but not analog recordings.
> >
> > I only have HD-3000 cards, so I can't really work on this aspect
> > of what I can see as being desired related functionality.
> >
> > On Tue, Apr 19, 2005 at 10:22:23PM -0400, Robert Tsai wrote:
> > > Today, HDTV recordings are limited to either no transcoding, or
> > > transcoding to the parameters in the MPEG2 recording profile.
> > > They aren't encoded at capture time, so there is no subsequent
> > > option to save a configurable amount of disk space.
> > >
> > > Attached is a patch to allow the configuration of a fixed number
> > > of transcoding profiles ("High Quality", "Medium Quality", and
> > > "Low Quality").
> > >
> > > Summary:
> > >
> > > - Upgrade DB schema:
> > >
> > > - The option to automatically transcode a recording
> > > has been moved out of the recording profile and into
> > > the record (so that the currently-unused
> > > "autotranscode" column is now useful).
> > >
> > > When the database is upgraded, recording schedules
> > > using recording profiles with auto-transcode enabled
> > > will have auto-transcode carried over.
> > >
> > > - Each record, in addition to a now-used
> > > "autotranscode" option, also gets a new "transcoder"
> > > option, which defaults to "Autodetect".
> > >
> > > - Delete the now-unused "autotranscode" option from
> > > the codecparams table.
> >
> > for non-"Live TV" profiles.
> >
> > > - Provide some fixed transcoders: "High Quality",
> > > "Medium Quality", and "Low Quality". Unfortunately, no
> > > default values are actually provided, the user must
> > > configure their desired values for these transcoders.
> > > Also unfortunately, there is no UI to add or remove
> > > transcoders (just like there is no UI to add or remove
> > > recording profiles). However, transcoders may be added
> > > to the database and things should work, if people need
> > > more than these three transcoders.
> > >
> > > - The default transcoder for all recordings is "0"
> > > (autodetect) to preserve current behavior.
> > >
> > > - Add some logging to indicate when transcoding is
> > > starting/stopping (both to the regular log as well as the
> > > database logs).
> > >
> > > - MythFrontend UI changes:
> > >
> > > - Add auto-transcode and transcoder options to each
> > > recording schedule, configurable at the time the
> > > recording is scheduled. Default values are:
> > >
> > > - No auto-transcode.
> > > - "Autodetect" transcoder.
> > >
> > > The "Autodetect" transcoder will choose between the
> > > "From MPEG4/RTjpeg" and "From MPEG2" transcoders,
> > > like it does today. Eventually the choice of
> > > transcoder can be made more configuration (e.g.,
> > > select transcoder based on 1080i/720p/480i
> > > resolution of recording). For now, just preserve
> > > behavior.
> > >
> > > - Add global settings UI to configure default values
> > > for auto-transcode and transcoder, near default values
> > > for commflagging and userjobs.
> > >
> > > - MythTranscode changes:
> > >
> > > - Fix a typo in the "help" text. :)
--Rob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mythtv.diff.bz2
Type: application/octet-stream
Size: 7288 bytes
Desc: not available
Url : http://mythtv.org/pipermail/mythtv-dev/attachments/20050502/ff6c337a/mythtv.diff.obj