On Mon, Mar 9, 2015 at 12:41 PM, Michael Niedermayer <michae...@gmx.at> wrote: > On Sun, Mar 08, 2015 at 11:07:33PM -0300, Claudio Freire wrote: >> On Sun, Mar 8, 2015 at 11:06 PM, Claudio Freire <klaussfre...@gmail.com> >> wrote: >> > On Sun, Mar 8, 2015 at 8:23 AM, Michael Niedermayer <michae...@gmx.at> >> > wrote: >> >> i think its better to leave fuzz at a small value otherwise we >> >> would forget to update the target when it improves and then >> >> subsequent worsenings could be missed >> > >> > An updated version with that fixed. >> > >> > It's a bit tricky to squash commits with git format-patch, so if >> > there's an issue with the patch just apply it with patch -p1, works >> > that way (tried) >> >> >> Dang, forgot to attach the patch. >> >> Now truly attached. > > works on x86 but fails on mips: > > stddev: 443.22 PSNR: 43.40 MAXDIFF: 5038 bytes: 176400/ 180224 > stddev: |443.22 - 414| >= 5 > Test aac-s7350-encode failed. Look at tests/data/fate/aac-s7350-encode.err > for details.
That's weird. I didn't expect this code to be platform-specific. Head is still using a butterworth filter for the lowpass IIRC. That could very well be both platform and sample rate specific. Sadly, I cannot test mips, but commenting the lowpass could yield useful results. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
