- Notifications
You must be signed in to change notification settings - Fork441
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 ourterms of service andprivacy 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?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
coveralls commentedDec 30, 2020 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
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. |
LSchmerber commentedSep 3, 2021
rico500 commentedAug 29, 2022
Has the development for this feature been discontinued? Is there a big road-block depending on another package (slycot?)? |
lytex commentedSep 20, 2022
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