Return-Path: From: Siarhei Siamashka To: hoene@uni-tuebingen.de Subject: Re: Testing SBC filtering functions Date: Mon, 29 Dec 2008 13:06:02 +0200 Cc: "'Marcel Holtmann'" , "'ext Brad Midgley'" , "'Jaska Uimonen'" , linux-bluetooth@vger.kernel.org References: <1227879337.20555.12.camel@esdhcp03999.research.nokia.com> <1230032888.8047.34.camel@californication> <001501c96996$20018070$60048150$@hoene@gmx.net> In-Reply-To: <001501c96996$20018070$60048150$@hoene@gmx.net> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Message-Id: <200812291306.02281.siarhei.siamashka@nokia.com> List-ID: On Monday 29 December 2008 11:16:23 ext Christian Hoene wrote: > Hello all, > > Merry Christmas! > > I wrote this script to make some tests on the SBC decoder and encoder using > the recommended testing procedure with the reference bitstreams, the > reference codec and PEAQ. > http://net.cs.uni-tuebingen.de/html/nexgenvoip/html/ > I got little bit confused with all the latest patches and whether they are > included or not. Just send me an email on which patch you like to have > tested. Running the tests just takes half an hour; me to answer my emails > maybe a bit longer. Please try the following patch (apply it to the latest git): http://marc.info/?l=linux-bluetooth&m=123054787830678&w=2 You can try the patch "as is", and also with SBC_HIGH_PRECISION define uncommented. High precision mode is naturally more likely to pass the conformance tests. I used my own script for testing with 'tiny_psnr' tool for comparing original file before encoding and the final decoded result (it measures standard deviation and PSNR). It would be interesting to see how our results correlate. According to my tests, encoder should now be OK with this patch. Decoder is definitely in a better shape than the current encoder from git. But the decoder also seems to have bugs which degrade quality significantly in some cases. Best regards, Siarhei Siamashka