[FFmpeg-devel] [PATCH] Apple RPZA encoder
Mon Mar 30 06:47:43 CEST 2009
On 3/25/09, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Tue, Mar 24, 2009 at 03:39:11PM +0530, Jai Menon wrote:
> > On 3/24/09, Jai Menon <jmenon86 at gmail.com> wrote:
> > > Hi,
> > >
> > > Attached patch is a cleaned up version of the original one posted by
> > > Todd Kirby .
> > > And, yeah, its a gsoc qualification task :)
> > >
> > >
> > >  http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/2005-June/001673.html
> > the isom.c hunk is obviously not required, sorry for the confusion.
> > correct patch attached.
> > +/* tuning parameters */
> > +#define SIXTEEN_COLOR_THRESH 24
> > +#define SKIP_FRAME_THRESH 13
> > +#define START_ONE_COLOR_THRESH 8
> > +#define CONTINUE_ONE_COLOR_THRESH 0
> Encoders, especially such simple ones should be build to be RD optimal
> in as many aspects as possible
The "parameters" so to say, give a good tradeoff between final output
size and quality. But I don't think I understand correctly. If we were
to use 16 colors everywhere, the output size will be absurdly large.
Other fixes were made, patch attached.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 28550 bytes
Desc: not available
More information about the ffmpeg-devel