[Octave-bug-tracker] [bug #50820] [octave forge] (signal) pwelch incorrectly discards DC component

[Mike Miller]

Update of bug #50820 (project octave):

                Severity:              3 - Normal => 2 - Minor              
                Priority:              5 - Normal => 1 - Later              
              Item Group:        Incorrect Result => Matlab Compatibility   
                  Status:                    None => Postponed              
                 Release:                   4.2.1 => other                  

    _______________________________________________________

Follow-up Comment #3:

Octave's pwelch already supports updated Matlab defaults, but via a strange
"R11" or "R12" argument passed in.

I haven't really modified pwelch myself, but it might make sense to try to
simplify all the code that tries to be compatible with different versions of
Matlab and just go with the latest. It's hard enough keeping up with the
changes that Matlab makes, but trying to keep code optionally backwards
compatible with older versions doesn't seem worth it.

So someone might want to look at the "compatib" internal variable in pwelch.m
and the various interpretations of positional arguments that is already done,
and choose whichever is closest to the current Matlab way.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?50820>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/