On Mon, 07 Nov 2011 10:16:39 -0800, Carnë Draug <carandraug+...@gmail.com>  
wrote:

> On 7 November 2011 16:43, Alberto Cuadrado <a.cuadr...@gmail.com> wrote:
>> Hello,
>>
>> I think there is a bug in the fir1 function in signal-1.0.11 (and  
>> probably
>> also in 1.1.1 because I don't see significant changes in the code). The
>> problem is that single-band lowpass filters returned by fir1 don't have  
>> a DC
>> gain of 1. For example:
>>
>> octave-3.2.4:1> sum(fir1(240,0.01))
>> ans =  1.1822
>>
>> That should return 1 (with low-order filters the problem is not so  
>> obvious).
>>
>> Regards
>>
>> Alberto Cuadrado
>
> I can confirm that this still occurs in octave 3.4.3 with signal 1.1.
> Do you think you can contribute a patch to fix this?
>


Are you sure it is a bug? I am assuming that you want the filter to be  
normalized the same way as Matlab's fir1 function, and they say:

    "By default, the filter is scaled so that the center of the first  
passband has a magnitude of exactly 1 after windowing."

In other words, they are normalizing so the *frequency domain* passband  
comes out to 1. The d.c. output level divided by the bandwidth will be  
(roughly) 1.

Regards,
Jeff Cunningham

------------------------------------------------------------------------------
RSA(R) Conference 2012
Save $700 by Nov 18
Register now
http://p.sf.net/sfu/rsa-sfdev2dev1
_______________________________________________
Octave-dev mailing list
Octave-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/octave-dev

Reply via email to