[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-bug-tracker] [bug #42859] bugs in periodogram.m
From: |
Rik |
Subject: |
[Octave-bug-tracker] [bug #42859] bugs in periodogram.m |
Date: |
Thu, 07 Aug 2014 17:33:24 +0000 |
User-agent: |
Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:31.0) Gecko/20100101 Firefox/31.0 |
Update of bug #42859 (project octave):
Status: None => Ready For Test
_______________________________________________________
Follow-up Comment #2:
periodogram was badly in need of updating. I made your changes as well as
overhauling the documentation, simplifying the input argument parsing, and
adding input validation tests in this changeset
(http://hg.savannah.gnu.org/hgweb/octave/rev/8ac4ab4ae5f4).
If someone has access to the Matlab signal toolbox could they try
pxx = periodogram (1:8, [], 4)
According to what I can decipher, Octave will get this wrong by a factor of 2
because it doesn't adjust 'n', the number of rows in X, after it downsized X
to match NFFT with this statement
x = sum (reshape (x, nfft, []), 2)
The function also needs some functional tests badly. This was a major rewrite
and I had to guess that everything worked out because there are no built-in
%!tests.
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/bugs/?42859>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/
- [Octave-bug-tracker] [bug #42859] bugs in periodogram.m,
Rik <=