[mythtv] Re: [PATCH] transcoding profiles v3

[Robert Tsai]

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:
> > 
> > 		- Remove "automatically transcode" option from
> > 		  recording profiles; auto-transcode is now a
> > 		  per-recording option (parallel with per-recording
> > 		  recording profiles).
> > 
> > 		- 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: 6120 bytes
Desc: not available
Url : http://mythtv.org/pipermail/mythtv-dev/attachments/20050501/3bd50b33/mythtv.diff.obj