[FFmpeg-devel] [PATCH] Apple RPZA encoder

[Jai Menon]

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 [1].
>  > >  And, yeah, its a gsoc qualification task :)
>  > >
>  > >
>  > >  [1] 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.

-- 
Regards,

Jai
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rpzaenc.patch
Type: text/x-patch
Size: 28550 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090330/40e58194/attachment.bin>