Skip to content

ctrlutil.issys() is confusing (tests for LTI) #876

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

Jump to bottom
Closed
murrayrm opened this issue Mar 25, 2023 · 0 comments · Fixed by #880
Closed

ctrlutil.issys() is confusing (tests for LTI) #876

murrayrm opened this issue Mar 25, 2023 · 0 comments · Fixed by #880
Assignees
@murrayrm
Milestone
0.9.4

Comments

@murrayrm
Copy link
Member

The function ctrlutil.issys() checks to see if a system is an LTI system. This probably made sense before we had I/O systems, but now it is confusing. It doesn't appear that issys is used anywhere, so I think we can deprecate it.
@murrayrm murrayrm mentioned this issue Mar 25, 2023
Merged
@murrayrm murrayrm added this to the 0.9.4 milestone Mar 26, 2023
@murrayrm murrayrm self-assigned this Mar 26, 2023
@murrayrm murrayrm mentioned this issue Mar 27, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@murrayrm murrayrm

Labels
None yet
Projects
None yet
Milestone
0.9.4
Development

Successfully merging a pull request may close this issue.

Small fixes for 0.9.4 release murrayrm/python-control
1 participant
@murrayrm