Hi. Can I improve something here? On Fri, Mar 30, 2018 at 2:43 AM, Rostislav Pehlivanov <atomnu...@gmail.com> wrote:
> On 30 March 2018 at 00:12, Даниил Чередник <dan.chered...@gmail.com> > wrote: > > > Actually this test does not make a sense since encoder now use floating > > point math. I should have noticed it before. > > Both patches have been updated. Checked on arm. > > > > On Thu, Mar 29, 2018 at 1:52 AM, Michael Niedermayer > > <mich...@niedermayer.cc > > > wrote: > > > > > On Tue, Mar 27, 2018 at 02:26:45AM +0300, Даниил Чередник wrote: > > > > Now it should be fine. Second patch has been fixed. > > > > > > > > On Mon, Mar 26, 2018 at 3:59 PM, Даниил Чередник < > > > dan.chered...@gmail.com> > > > > wrote: > > > > > > > > > Hi. > > > > > Yes, I reproduced it on mac. A bit strange, I run fate with > valgrind > > > > > during patch preparation, it was ok. > > > > > I will fix it, and send updated patch. > > > > > > > > > > Thank you! > > > > > > > > > > On Sun, Mar 25, 2018 at 5:57 PM, Michael Niedermayer < > > > > > mich...@niedermayer.cc> wrote: > > > > > > > > > >> On Sun, Mar 25, 2018 at 01:55:42PM +0300, Даниил Чередник wrote: > > > > >> [...] > > > > >> > > > > >> > libavcodec/dcaenc.c | 369 ++++++++++++++++++++---------- > > > > >> -------------------- > > > > >> > libavcodec/psymodel.c | 1 > > > > >> > tests/fate/acodec.mak | 4 > > > > >> > 3 files changed, 156 insertions(+), 218 deletions(-) > > > > >> > fd146632a7f29530a59d35abd63149c81b4dfcc6 > > > > >> 0002-avcodec-dcaenc-Use-aac-psychoacoustic-model-for-DCA-.patch > > > > >> > From 99d937a0828bbd60aef52d7979c75f8c21989145 Mon Sep 17 > 00:00:00 > > > 2001 > > > > >> > From: Daniil Cherednik <dan.chered...@gmail.com> > > > > >> > Date: Sun, 4 Mar 2018 13:14:17 +0000 > > > > >> > Subject: [PATCH 2/2] avcodec/dcaenc: Use aac psychoacoustic > model > > > for > > > > >> DCA > > > > >> > encoder > > > > >> > > > > > >> > There are several reasons to replace dca psychoacoustic to > common > > > model: > > > > >> > - dca psychoacoustic has some quality problems especially at > high > > > > >> frequency bands > > > > >> > - unclear implementation > > > > >> > - aac implementation allows to use tonality measurement for > > future > > > > >> improvements > > > > >> > - a bit faster > > > > >> > > > > >> This breaks > > > > >> make -j12 fate-acodec-dca > > > > >> (segfaults) > > > > >> > > > > >> tell me if you can reproduce? If not ill rebuild with debug > symbols > > > and > > > > >> get a > > > > >> backtrace > > > > >> > > > > >> [...] > > > > >> -- > > > > >> Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC7 > > > 87040B0FAB > > > > >> > > > > >> If you drop bombs on a foreign country and kill a hundred thousand > > > > >> innocent people, expect your government to call the consequence > > > > >> "unprovoked inhuman terrorist attacks" and use it to justify > > dropping > > > > >> more bombs and killing more people. The technology changed, the > idea > > > is > > > > >> old. > > > > >> > > > > >> _______________________________________________ > > > > >> ffmpeg-devel mailing list > > > > >> ffmpeg-devel@ffmpeg.org > > > > >> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > > > >> > > > > >> > > > > > > > > > > > > > > > -- > > > > > Daniil Cherednik > > > > > > > > > > > > > > > > > > > > > -- > > > > Daniil Cherednik > > > > > > [...] > > > > > > > 57b2e6d001a856c9434fe6ad1856cb3f24443a83 > 0002-avcodec-dcaenc-Use-aac- > > > psychoacoustic-model-for-DCA-.patch > > > > From 49b4f2a966719c24f54c026c438680557c35e96b Mon Sep 17 00:00:00 > 2001 > > > > From: Daniil Cherednik <dan.chered...@gmail.com> > > > > Date: Sun, 4 Mar 2018 13:14:17 +0000 > > > > Subject: [PATCH 2/2] avcodec/dcaenc: Use aac psychoacoustic model for > > DCA > > > > encoder > > > > > > > > There are several reasons to replace dca psychoacoustic to common > > model: > > > > - dca psychoacoustic has some quality problems especially at high > > > frequency bands > > > > - unclear implementation > > > > - aac implementation allows to use tonality measurement for future > > > improvements > > > > - a bit faster > > > > > > > > Signed-off-by: Daniil Cherednik <dan.chered...@gmail.com> > > > > --- > > > > libavcodec/dcaenc.c | 369 +++++++++++++++++++++--------- > > > -------------------- > > > > libavcodec/psymodel.c | 1 + > > > > tests/fate/acodec.mak | 4 +- > > > > 3 files changed, 156 insertions(+), 218 deletions(-) > > > > > > breaks fate on arm qemu > > > > > > TEST acodec-dca > > > --- - 2018-03-29 00:37:01.286083392 +0200 > > > +++ tests/data/fate/acodec-dca 2018-03-29 00:37:01.283607451 +0200 > > > @@ -1 +1 @@ > > > -c610f911aa74656132abb650be6b548a > > > +503d8eac7238d87aae6f92808231e0c0 > > > Test acodec-dca failed. Look at tests/data/fate/acodec-dca.err for > > details. > > > make: *** [fate-acodec-dca] Error 1 > > > > > > > > > [...] > > > > > > -- > > > Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC7 > 87040B0FAB > > > > > > No human being will ever know the Truth, for even if they happen to say > > it > > > by chance, they would not even known they had done so. -- Xenophanes > > > > > > _______________________________________________ > > > ffmpeg-devel mailing list > > > ffmpeg-devel@ffmpeg.org > > > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > > > > > > > > > > > -- > > Daniil Cherednik > > > > _______________________________________________ > > ffmpeg-devel mailing list > > ffmpeg-devel@ffmpeg.org > > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > > > > I'll take a look at it tomorrow, thanks > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > -- Daniil Cherednik _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
