-
Notifications
You must be signed in to change notification settings - Fork 438
allow complex statespace matrices #484
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
base: main
Are you sure you want to change the base?
Conversation
I suspect this might be quite a lot of work to properly support (see below). Would we change Slycot to reject complex matrices when real are expected? This might be a good idea anyway. Would we support TFs with complex coefficients?
|
Also, see discussion in #376. We could set a warning filter making this an error in Slycot. |
Has the development for this feature been discontinued? Is there a big road-block depending on another package (slycot?)? |
If I remember correctly, Slycot had some functions that could be easily extended to handle complex cases
|
The main roadblock is probably developer time. I might resuming working on this not earlier than in 2 or 3 months. Contributions from others are always welcome of course. |
Outsourced from #438, because it is not directly related to the test suite.
To be combined with #376