GeezerVison is showing github.com — add zpk() function by murrayrm · Pull Request #816 · python-control/python-control
Conversation
murrayrm
commented
Dec 16, 2022
•
edited
Loading
Uh oh!
There was an error while loading. Please reload this page.
This PR adds the function zpk, which creates a transfer function (as a NamedIOSystem) from a zero, pole, gain representation. This responds to issue #749.
Also added in code to make sure that signal names are copied correctly in various conversions, removing some comments entered in PR #804. In _convert_to_transfer_function this involved removing renaming from the low level function, as documented in the comment string.
Notes:
-
Unlike in MATLAB and SciPy, which have zpk classes, here we just create a transfer function from the zpk data. That seems like the most common use case.
-
In the matlab module there are also functions zpk2ss and zpk2tf that implement lower level behavior (eg, zpk2ss returns A, B, C, D). This is similar to MATLAB. if you want to get a StateSpace system from a zpk representation, use sys = ct.ss(ct.zpk(zeros, poles, gain)).
-
Unit tests and updated documentation are also included.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
coveralls
commented
Dec 16, 2022
•
edited
Loading
Uh oh!
There was an error while loading. Please reload this page.
|

Coverage increased (+0.006%) to 94.817% when pulling 7ee0c0e on murrayrm:zpk-13Dec2022 into b4cd96b on python-control:main.
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Assignees
No one assigned
Development
Successfully merging this pull request may close these issues.
Include a zpk2tf function
Uh oh!
There was an error while loading. Please reload this page.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.