[Octave-bug-tracker] [bug #51310] [octave forge] (signal) firls.m modification to include all 4 FIR types, Hilbert transformer, and differentiator

[Mike Miller]

Follow-up Comment #18, bug #51310 (project octave):

Thank you, I will take a look at this latest revision in the next week.

I sincerely appreciate all the work you have put into this function
improvement.

I understand and appreciate the learning curve when submitting changes,
especially when there are code style guidelines and conventions that you may
not be familiar with. We try to document these things for new contributors,
but we could use some help improving that. There's no need to apologize, our
community should do a better job of documenting and supporting new
contributors.

If you are talking about other functions in the signal package, yes, many of
these files are 10 or 20 years old now, from before a lot of the Octave coding
guidelines have been applied. As a background task I have been working to make
these older functions conform to the current coding guidelines, but it's a
slow process. If you are looking for functions to compare with, it's far
better to look at Octave itself, not the signal package, as a standard.

    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?51310>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/