patchGNU Octave - Patches: patch #9265, [octave forge] (signal) isstable:...

 
 

patch #9265: [octave forge] (signal) isstable: new function

Submitter:  -X- <vlefkopo>
Submitted:  Sun 19 Feb 2017 07:37:32 PM UTC
   
 
Category:  Forge : new function Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  lostbard Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 17 May 2023 05:52:57 PM UTC, comment #5: 

v1.4.4 released - closing ticket

John Donoghue <lostbard>
Group Member
Mon 08 May 2023 05:59:04 PM UTC, comment #4: 

Pushed to repo

John Donoghue <lostbard>
Group Member
Sun 24 Apr 2022 01:55:05 PM UTC, comment #3: 

Using version 6.1.0
Control 3.2.0
Signal 1.4.2

All tests work

>> test isstable
PASSES 4 out of 4 tests


Updated Vasilis' instructions to octave style.

(file #53140)

Anonymous
Mon 05 Apr 2021 05:31:50 AM UTC, comment #2: 

Since this has not been touched.
Improved the docstrings format
Then did the tests

@Mike
Has something changed?


>> [z,p,g] = butter(6,0.7,'high');
>> sos = zp2sos(z,p,g);
error: cplxpair: could not pair all complex numbers
error: called from
    cplxpair at line 118 column 9
    cplxreal at line 66 column 7
    zp2sos at line 79 column 10



My versions:
octave 4.2.2 (This fits the time of your test)
control 3.1
signal 1.4

Is this a bug or am I doing something wrong?

Anonymous
Fri 30 Mar 2018 08:36:05 PM UTC, comment #1: 

Thank you for your work on this function so far.

The function works for me, looks fairly Matlab compatible, and includes unit tests that all pass. Very good work.

The coding style mostly conforms to the Octave coding style, but could use some small improvements. Octave always includes a space between the function name and the parenthesized argument list. The function doc string should be written in Texinfo format. And some more input validation could be done to ensure the arguments are correct before they are used.

I can help clean some of these things up if you don't have time or interest in working on this. Let me know if you would like to update the function file.

Mike Miller <mtmiller>
Group Member
Sun 19 Feb 2017 07:37:32 PM UTC, original submission:  

I implemented the isstable function of the signal package.

Since this is my first time submitting, I would be grateful for any kind of feedback and comments that would benefit my future work and submissions.

-X- <vlefkopo>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #53140:  isstable.m added by None (3KiB - application/octet-stream)
file #39791:  isstable.m added by vlefkopo (2KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by lostbard (Posted a comment)
  • -email is unavailable- added by vlefkopo (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 10 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2023-05-17 lostbard StatusReady For Test Done
        Open/ClosedOpen Closed
    2023-05-08 lostbard StatusNeed Info Ready For Test
        Assigned tomtmiller lostbard
    2022-04-24 None Attached File- Added isstable.m, #53140
    2019-03-07 mtmiller Carbon-CopyRemoved 80942 -
    2018-03-30 mtmiller StatusNone Need Info
    2018-03-30 mtmiller Assigned toNone mtmiller
        Summarysignal package: Implemented isstable function [octave forge] (signal) isstable: new function
    2017-02-19 vlefkopo Attached File- Added isstable.m, #39791

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code