[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-patch-tracker] [patch #9199] [octave forge] (signal) statelevels
|
From: |
Mike Miller |
|
Subject: |
[Octave-patch-tracker] [patch #9199] [octave forge] (signal) statelevels: new function |
|
Date: |
Fri, 30 Mar 2018 16:27:41 -0400 (EDT) |
|
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:59.0) Gecko/20100101 Firefox/59.0 |
Update of patch #9199 (project octave):
Status: None => Need Info
_______________________________________________________
Follow-up Comment #3:
Thanks for the work you have put into this function so far.
This function is needed, looks functionally correct and Matlab compatible to
me.
It would be very helpful if you could add some unit tests to the end of the
file. It would also be helpful to conform to the Octave coding style, variable
naming conventions, and include a descriptive doc string.
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/patch/?9199>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/