Skip to content

Feature print zpk #869

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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Merged
murrayrm merged 16 commits into from
Mar 25, 2023
Merged

Feature print zpk #869

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
5c85459
Add print zpk form
henklaak Feb 21, 2023
57620f1
Fix unittest
henklaak Feb 21, 2023
b36732d
Add fix for precision noise
henklaak Feb 21, 2023
45b5ae4
Cleanup unittest
henklaak Feb 21, 2023
07a859d
Merge branch 'python-control:main' into feature_print_zpk
henklaak Feb 24, 2023
93f7b46
Rework after review
henklaak Feb 24, 2023
d2807b0
Shouldn't have commited the pyproject.toml, revering
henklaak Feb 24, 2023
5652bf0
Fix variable mixup due to inconsistent naming
henklaak Feb 24, 2023
58e9d63
Fix variable mixup due to inconsistent naming
henklaak Feb 24, 2023
521a306
Typos
henklaak Feb 24, 2023
2644874
Add default for xferfcn.display_format
henklaak Feb 25, 2023
8101b31
Update after failed tests
henklaak Feb 25, 2023
8d7ac56
Add default for xferfcn.floating_point_format
henklaak Feb 25, 2023
2ca47e1
Fix improper way of setting defaults in test
henklaak Feb 25, 2023
5f506b2
Merge branch 'python-control:main' into feature_print_zpk
henklaak Mar 5, 2023
6933973
update xferfcn._float2str to be more pythonic per @ilayn
murrayrm Mar 25, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix unittest
  • Loading branch information
@henklaak
henklaak committed Feb 21, 2023
commit 57620f19b742ef40cdeb9b7cc4e4da1977383ddd
6 changes: 5 additions & 1 deletion control/xferfcn.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -505,9 +505,13 @@ def to_zpk(self, var=None):
if abs(polygain) == 0 and abs(dcgain) == 0:
k = 1
else:
if abs(polygain) == 0:
raise ValueError(
f"Transfer function has infinite gain. "
"Please check polynomials.")
k = dcgain/polygain
if not np.isreal(k):
raise ValueError("Transfer function has complex valued gain. "
raise ValueError(f"Transfer function has complex valued gain (k = {k}). "
"Please check polynomials for non-complimentary poles.")
Comment thread
henklaak marked this conversation as resolved.
Outdated

k = np.abs(k)
Expand Down