Re: [LAD] Aeolus

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: hermann meyer <brummer-@...>
Cc: <linux-audio-dev@...>
Date: Friday, September 20, 2013 - 3:58 pm

On Thu, Sep 19, 2013 at 07:07:37PM +0200, hermann meyer wrote:

> I've attached the diff, for your review.

Thanks. It would seem possible to merge the two versions, but
I have a few questions/remarks.

1. You shouldn't have removed the 'options'. In the current release
they are used for things not related to the FFT library, and other
options may be added in the future. It has no performance penalty.

2. You have changed the gain normalisation from 0.5 / partition to
1.0 / partition. Why ? The 0.5 is not related to anything specific
FFTW. It's there because the FFT size is twice the partition size.
The result should be that convolution with a single impulse of 1.0
amplitude reproduces the input without a gain change.

3. If av_malloc() can ever fail, this should raise an exception.

4. Assuming that the 'swap' operation that is part of the av FFT
only changes the order of complete complex values (and not of
their components), all the swaps can be removed. The only thing
that happens in the F-domain is that values with the same index
are multiplied. Their order is not important.

Ciao,

--
FA

A world of exhaustive, reliable metadata would be an utopia.
It's also a pipe-dream, founded on self-delusion, nerd hubris
and hysterically inflated market opportunities. (Cory Doctorow)

_______________________________________________
Linux-audio-dev mailing list
Linux-audio-dev@lists.linuxaudio.org
http://lists.linuxaudio.org/listinfo/linux-audio-dev

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[LAD] Aeolus, Fons Adriaensen, (Wed Sep 18, 9:16 pm)
Re: [LAD] Aeolus, James Morris, (Thu Sep 19, 2:36 pm)
Re: [LAD] Aeolus, Ralf Mardorf, (Fri Sep 20, 12:55 am)
Re: [LAD] [LAU] Aeolus, Julien Claassen, (Wed Sep 18, 10:02 pm)
Re: [LAD] Aeolus, J. Liles, (Wed Sep 18, 9:29 pm)
Re: [LAD] Aeolus, Fons Adriaensen, (Wed Sep 18, 10:29 pm)
Re: [LAD] Aeolus, hermann meyer, (Thu Sep 19, 3:31 am)
Re: [LAD] Aeolus, IOhannes m zmoelnig, (Thu Sep 19, 2:50 pm)
Re: [LAD] Aeolus, hermann meyer, (Thu Sep 19, 3:18 pm)
Re: [LAD] Aeolus, IOhannes m zmoelnig, (Thu Sep 19, 3:50 pm)
Re: [LAD] Aeolus, hermann meyer, (Thu Sep 19, 4:57 pm)
Re: [LAD] Aeolus, IOhannes m zmölnig, (Thu Sep 19, 8:48 pm)
Re: [LAD] Aeolus, hermann meyer, (Thu Sep 19, 7:00 pm)
Re: [LAD] Aeolus, Fons Adriaensen, (Thu Sep 19, 3:36 pm)
Re: [LAD] Aeolus, hermann meyer, (Thu Sep 19, 5:07 pm)
Re: [LAD] Aeolus, Fons Adriaensen, (Fri Sep 20, 3:58 pm)
Re: [LAD] Aeolus, hermann meyer, (Fri Sep 20, 4:51 pm)
Re: [LAD] Aeolus, Gene Heskett, (Thu Sep 19, 3:14 pm)
Re: [LAD] [LAU] Aeolus, Rafael Vega, (Wed Sep 18, 10:40 pm)
Re: [LAD] Aeolus, Nils Gey, (Wed Sep 18, 9:36 pm)
Re: [LAD] Aeolus, Dan Muresan, (Wed Sep 18, 9:46 pm)