[mythtv] [PATCH] Transcoder w/cutlist updates

[Geoffrey Hausheer]

On Mon, 16 Jun 2003 15:01:40 -0400, "Craig Longman craigl-at-begeek.com
|mythtv/1.0-Allow|" <sa9nhe32c40t at sneakemail.com> said:
> Geoffrey Hausheer wrote:
> 
> >Stuff on my plate to do still:
> >* Don't allow a manual-transcode if the cutlist is empty.
> >
> why is this?  i might want to simply re-encode something down, but want 
> the whole thing.

Well, today you can only do manual transcoding if the
'TranscoderuseCutlist' option is set (okay, well today, you can't
manually transcode, but assuming the patch goes in (and once the polling
thread is enabled), this will be the case).  The reason for this is that
if 'TranscoderUseCutlist' is disabled, the transcoding will happen
automatically as soon as the recording is done, so in its current form,
it makes no sense to transcode if the cutlist is empty.  I hope to find a
better solution to this (possibly an 'autoTranscode' setting).  The point
is to prevent continuosly transcodingthe same file, since currently, we
will lose quality in each pass.  In reality, my statement probably should
have read:

* Don't transcode if there is nothing to do
(i.e. the source and destination have the same format, and the cutlist
does not require any changes)

I'd actually like other opinions on how people would prefer this would
work.  I'd like to not have lots of transcoding options if they aren't
needed, so I can keep the transcoder as easy to setup as possible (once
it is all working, I figured the defualt would be 1 transcoder with
use-cutlist enabled, and no auto-transcoding)

> 
> >* Process cutlist without reencoding the entire file.
> >
> there is definitely going to be some cross-over between what i'm doing 
> and this, we should talk when you get to this point so i can give you 
> what i've done, if it hasn't already been submitted/accepted.
I agree.  I have looked at it a bit, and I don't think the changes are
too invasive (DoFastForward needs some modifications, and I need to be
able to update the frame numbers in the frame-headers), but this will
change ReencodeFile some, and I know that is what you are working on.

On the other hand, I'd like to get the transcoder up to snuff as soon as
possible so people can actually start using it.  I like keeping
development open, so I prefer posting my patches to the dev-list (that
way if people have an issue with them, they can let me know); we are only
talking about a few functions here, and dealing with conflicts shouldn't
be that difficult.  Anyhow, if you like, I'll send you a patch once I
have it mostly working so you can let me know of any issues you have
before I am finished with it.  Alternatively, feel free to post what
you've got so far and I'll either incorporate it, or try to minimize the
conflicts.

Thanks,
.Geoff
-- 
  Geoffrey Hausheer
  XXXXXXXXXXXXXXXXXXXXX