[FFmpeg-soc] [PATCH] AMR-WB Decoder

[Vitor Sessak]

On 09/06/2010 02:46 AM, Marcelo Galvão Póvoa wrote:
> Ok, fortunately I've found the bug!
>
> It was just a MIN_ISF_SPACING parameter which I extracted from the
> reference code but was unsure about it's Q level. After some time, I
> thought I have it figured out but I was wrong. Now I know the answer
> the hard way...
>
> The clipping and the sharp peaks are gone, the waveforms are really
> close now!

That's great news!

> Also, the stddev against the reference decoder decreased a
> lot (it was ~884 before):
> all_men.awb stddev:   51.72 PSNR: 62.05 MAXDIFF: 1089 bytes:   473600/   473600
>    

stddev of 51 looks pretty good to me for this case.

> So, here is how the decoder is now (I'm sending my commits to github also).
>
> It is still missing the bitstream_unpack() function sharing with
> AMR-NB which I will do soon.
>    

I'll try to review some more your code as soon as I have some time.

-Vitor