Hi Nitzan,

Apologies. I'm responsible for that bug.
You have a good fix.
I'll take care of it tomorrow if it's not done before then.
Thanks,
Peter Lanspeary

On Thu, Mar 26, 2009 at 05:01:14PM +0200, nit ar wrote:
> 
> Hello octaveforge developers
> 
> I have found a bug in pwelch.m for the case when a cross power spectrum is 
> calculated and 'none' is passed as a parameter in order to remove mean from 
> both x and y inputs:
> 
> line 718:
>       elseif ( rm_mean ~= 2 ) % remove linear trend from segment
> 
> should be chamged to:
>       elseif ( rm_mean == 2 ) % remove linear trend from segment
> 
> Treatment of y should be equivalent to the treatment of x (line 707).
> The above change produce results which are compatible with matlab.
> 
> Regards
> Nitzan Arazi 

------------------------------------------------------------------------------
_______________________________________________
Octave-dev mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/octave-dev

Reply via email to