[Octave-patch-tracker] [patch #8535] [signal] firpm (new remez)

[Laurent Goulet]

Follow-up Comment #5, patch #8535 (project octave):

The update looks pretty good. I was a bit surprised at the relative weight of
the passband vs. stopband. 
Not much of an issue since the user can provide weights to obtained whatever
passband/stopband tradeoff they desire, but to get the same results as remez
(and some literature results) without playing with the weights, you'd actually
have to tweak W(f) to divide by (f/2) instead of f. 
I guess the question is what does Matlab generate in that case (if we want to
match Matlab's output for the same arguments). I unfortunately don't have
Matlab to check.

Strangely enough the issue with VS (which I reproduced with VS2010, 2012 and
2013 express) messes up the second argument to the max macro, and it's still
an issue with the renamed max_ variable
(see [this microsoft connect
issue](https://connect.microsoft.com/VisualStudio/feedback/details/1101733/incorrect-nested-macro-expansion)).


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?8535>

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