Skip to content

Keyword argument checking #713

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
bnavigator merged 8 commits into from
Mar 30, 2022
Merged

Keyword argument checking #713

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
8b6bfd1
check for unused keywords
murrayrm Mar 20, 2022
3656968
remove_useless -> remove_useless_states in iosys
murrayrm Mar 20, 2022
ceddc6f
add test for extraneous keywords in iosys
murrayrm Mar 20, 2022
97a0a14
add test for extraneous keywords in xferfcn
murrayrm Mar 20, 2022
57e3751
add test for extraneous keywords in statesp
murrayrm Mar 20, 2022
a86d33f
add kwargs_test + checks for unrecognized keywords
murrayrm Mar 21, 2022
26303ff
move kwarg lists inside function to fix initialization issue
murrayrm Mar 21, 2022
87cb31a
add missing kwarg tests + small bug fixes
murrayrm Mar 22, 2022
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
add test for extraneous keywords in statesp
  • Loading branch information
@murrayrm
murrayrm committed Mar 30, 2022
commit 57e3751ffabc5385898eaaaa6576431906ccbd3a
5 changes: 5 additions & 0 deletions control/statesp.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1776,7 +1776,12 @@ def _ss(*args, keywords=None, **kwargs):
"""Internal function to create StateSpace system"""
if len(args) == 4 or len(args) == 5:
return StateSpace(*args, keywords=keywords, **kwargs)

elif len(args) == 1:
# Make sure there were no extraneous keywords
if kwargs:
raise TypeError("unrecognized keywords: ", str(kwargs))

from .xferfcn import TransferFunction
sys = args[0]
if isinstance(sys, StateSpace):
Expand Down