[FFmpeg-devel] AMR-NB decoder

Benoit Fouet benoit.fouet
Thu Aug 13 14:33:28 CEST 2009


On 2009-08-13 10:38, Colin McQuillan wrote:
> 2009/8/10 Colin McQuillan <m.niloc at googlemail.com>:
>> 2009/8/6 Michael Niedermayer <michaelni at gmx.at>:
>>> On Wed, Aug 05, 2009 at 05:51:36PM +0100, Colin McQuillan wrote:
>>>> Attached is a patch for an AMR-NB decoder.
>>>>
>>>> It is not bit-exact. This makes it tricky to verify, but I have been
>>>> checking that internal parameters match the 3GPP decoder for the AMR
>>>> test sequences. The PSNR between the input and output is 3.90 to 8.42
>>>> which is about the same as the reference decoder. The PSNR between the
>>>> two outputs is between 8.50 and 18.16, which seems quite good.
> 
> [...]
> 
>> I have tried to address every other comment made in the thread. The
>> main changes are cosmetics, using fixed point tables to save space,
>> and keeping the algebraic codebook vector sparse until
>> anti_sparseness.
>>
>> amrnb-decoder-r2.patch uses high-pass-filterf.patch, agc-3.patch, and
>> circ-addf.patch.
> 
> Any comments on that patch? It needs to be updated but I don't want to
> send out 200KB patches every time I change a function name.

can't we get the tables checked in ?

Ben



More information about the ffmpeg-devel mailing list