Skip to content

squeeze: return shape of the frequency response. #594

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
bnavigator opened this issue Mar 28, 2021 · 3 comments · Fixed by #595
Closed

squeeze: return shape of the frequency response. #594

bnavigator opened this issue Mar 28, 2021 · 3 comments · Fixed by #595

Comments

@bnavigator
Copy link
Contributor

bnavigator commented Mar 28, 2021 •
edited
Loading

Something strange about the return shape of the frequency response. For a square 2x2 system:

sys(np.array(0.0), squeeze=False).shape
(2, 2, 1)

sys(0.0, squeeze=False).shape
(2, 2)

Originally posted with the wrong result mapping by @forgi86 in #593 (comment)
This was referenced Mar 28, 2021
Merged
Merged
@bnavigator
Copy link
Contributor Author

bnavigator commented Mar 28, 2021 •
edited
Loading

BTW, omega has to be complex.

>>> import numpy as np
>>> from control import rss
>>> sys = rss(2,2,2)
>>> sys(np.array(0.0j), squeeze=False).shape
(2, 2, 1)
>>> sys(0.0j, squeeze=False).shape
(2, 2)

Now I am even more confused and will stop for this Sunday evening.

@murrayrm
Copy link
Member

murrayrm commented Mar 28, 2021 •
edited
Loading

Sorry, accidentally closed.

@murrayrm murrayrm reopened this Mar 28, 2021
@forgi86
Copy link
Contributor

forgi86 commented Mar 29, 2021

@bnavigator as far as I can see looks like you can pass real/complex/imaginary argument. I would expect the output to be the Laplace/Z transform for ct/dt in general, but haven't really tested!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

lti squeeze: ndarray.ndim == 0 is also a scalar bnavigator/python-control
3 participants
@murrayrm @bnavigator @forgi86