[Octave-bug-tracker] [bug #42859] bugs in periodogram.m

[Rik]

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/