Skip to content
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.

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

squeeze: return shape of the frequency response. #594

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

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)

@bnavigator
Copy link
Contributor Author

bnavigator commented Mar 28, 2021

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

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
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants