Message ID | 20180328225226.GP32664@michaelspb |
---|---|
State | Not Applicable |
Headers | show |
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 <michael@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.cherednik@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 < > > > michael@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.cherednik@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.cherednik@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.cherednik@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: 9FF2128B147EF6730BADF133611EC787040B0FAB > > 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 > >
On 30 March 2018 at 00:12, Даниил Чередник <dan.cherednik@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 > <michael@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.cherednik@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 < > > > > michael@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.cherednik@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.cherednik@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.cherednik@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: 9FF2128B147EF6730BADF133611EC787040B0FAB > > > > 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
Hi. Can I improve something here? On Fri, Mar 30, 2018 at 2:43 AM, Rostislav Pehlivanov <atomnuker@gmail.com> wrote: > On 30 March 2018 at 00:12, Даниил Чередник <dan.cherednik@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 > > <michael@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.cherednik@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 < > > > > > michael@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.cherednik@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.cherednik@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.cherednik@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 >
--- - 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