octave-patch-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Octave-patch-tracker] [patch #9409] [octave forge] (signal) modulate: n


From: Mike Miller
Subject: [Octave-patch-tracker] [patch #9409] [octave forge] (signal) modulate: new function, partial implementation
Date: Wed, 10 Oct 2018 12:49:12 -0400 (EDT)
User-agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.100 Safari/537.36

Follow-up Comment #3, patch #9409 (project octave):

Hi Sudeepam, I haven't had a chance to review or test this yet in detail, but
I do notice that there are no test cases. Would you mind adding some simple
tests and/or demos to the end of the function? The indentation in the function
also needs to be cleaned up.

    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/patch/?9409>

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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]