NotificationsYou must be signed in to change notification settings
Fork441
Star1.8k

fix damp command natural frequency printout for discrete poles on real axis#894

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

murrayrm merged 3 commits intopython-control:mainfromsawyerbfuller:damp-dt-freq

May 28, 2023

Merged

fix damp command natural frequency printout for discrete poles on real axis#894

murrayrm merged 3 commits intopython-control:mainfromsawyerbfuller:damp-dt-freq

May 28, 2023

Conversation

Copy link

Contributor

sawyerbfuller commentedMay 22, 2023

Thedamp command was assuming a cont-time system when printing out pole locations (but the return value was correct). Fixed, along with making a few docstring improvements.

sawyerbfuller added3 commits

May 22, 2023 14:37

fix damp command natural frequency printout for discrete poles on rea…

47cbe72

…l axis; docstring fixes

docstring make explicit that doprint is optional

cec6b5c

output more readable

6cfd466

Copy link

ContributorAuthor

sawyerbfuller commentedMay 23, 2023

Example:

sysd=ct.zpk([], [-1.1,1+1j,1-1j],1,True)ct.damp(sysd)

Old (pole at -1.1 has the wrong frequency, table is hard to read):

_____Eigenvalue______ Damping___ Frequency_         1        +1j    -0.4037     0.8585         1        -1j    -0.4037     0.8585      -1.1                     1        1.1

New:

    Eigenvalue (pole)       Damping     Frequency         1        +1j       -0.4037        0.8585         1        -1j       -0.4037        0.8585                 -1.1             1         3.143