patchGNU Octave - Patches: patch #9256, damp.m - Implemented missing...

 
 

patch #9256: damp.m - Implemented missing function "damp" of the Octave-Forge control package

Submitter:  None
Submitted:  Thu 09 Feb 2017 09:42:37 PM UTC
   
 
Category:  Forge : new function Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  None Originator Email:  -email is unavailable-
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Tue 06 Mar 2018 02:59:46 AM UTC, comment #4: 

This can now be closed.
damp.m is now in control-3.1.0

Doug Stewart <dastew>
Tue 27 Jun 2017 09:17:36 PM UTC, comment #3: 

I don't want to be impatient or rude, but is anybody actually reviewing these files in the patch tracker?
I know this is an open source project and you are working on this in your free time, but I submitted this almost 5 months ago and reviewing and merging the code shouldn't take long.

It would be nice to get a short reply, so I know that it's worth the effort (since there are some more functions I would like to work on).

That said, I really enjoy Octave and its Control package!

--Mark

Anonymous
Sat 20 May 2017 12:52:44 PM UTC, comment #2: 

In the lastest file submission, I re-formatted the code to follow Octave's style guide more closely (only cosmetic changes). --Mark

Anonymous
Thu 09 Mar 2017 01:31:42 PM UTC, comment #1: 

Any news/feedback on this one?
Do you want me to change anything? It should be ready to be merged though...
Please note that one test will fail unless my other "pole.patch" submission (https://savannah.gnu.org/patch/index.php?9255) is applied.

-- Mark


Anonymous
Thu 09 Feb 2017 09:42:37 PM UTC, original submission:  

## Calculate natural frequencies, damping ratios and poles.
##
## If no output is specified, display overview table containing poles, magnitude
## (if @var{sys} is a discrete-time model), damping ratios, natural frequencies
## and time constants.
##
## @strong{Inputs}
## @table @var
## @item sys
## @acronym{LTI} model.
## @end table
##
## @strong{Outputs}
## @table @var
## @item Wn
## Natural frequencies of each pole of @var{sys} (in increasing order).
## The frequency unit is Hz (reciprocal of a second).
## If @var{sys} is a discrete-time model with specified sample time, @var{Wn}
## contains the natural frequencies of the equivalent continuous-time poles
## (see Algorithms).  If @var{sys} has an unspecified sample time
## (@var{tsam =} -1), @var{tsam =} 1 is used to calculate @var{Wn}.
## @item zeta
## Damping ratios of each pole of @var{sys} (in the same order as @var{Wn}).
## If @var{sys} is a discrete-time model with specified sample time, @var{zeta}
## contains the damping ratios of the equivalent continuous-time poles
## (see Algorithms).  If @var{sys} has an unspecified sample time
## (@var{tsam =} -1), @var{tsam =} 1 is used to calculate @var{zeta}.
## @item P
## Poles of @var{sys} (in the same order as @var{Wn}).

Anonymous

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #40757:  damp.m added by None (7KiB - text/x-objcsrc - One final change in code formatting to improve readability (only cosmetic). --Mark)
file #40744:  damp.m added by None (7KiB - text/x-objcsrc - Disregard the previous submissions. This file contains the newest version of the function, where I re-formatted the code to follow Octave's style guide more closely. --Mark)
file #39768:  damp.patch added by None (553B - text/x-patch - Corrected small error in help section (frequency is in rad/s, not Hz). --Mark)
file #39714:  damp.m added by None (7KiB - text/x-objcsrc)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by i7tiol (Updated the item)
  • -email is unavailable- added by dastew (Posted a comment)
  •  

    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 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-03-07 i7tiol StatusNone Done
        Open/ClosedOpen Closed
    2017-05-22 None Attached File- Added damp.m, #40757
    2017-05-20 None Attached File- Added damp.m, #40744
    2017-02-16 None Attached File- Added damp.patch, #39768
    2017-02-09 None Attached File- Added damp.m, #39714

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code