-
Notifications
You must be signed in to change notification settings - Fork 441
Conda-forge version of slycot 0.3.3 generates errors #217
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
Comments
This error occurs with conda forge's slycot 0.2.0, 0.3.0, and 0.3.3. |
How precise is this test? I can imagine minreal being finicky because of floating point operations. How robust is it to different platforms, architectures, and blas/lapack implementations? |
Good to know that this is not a new bug. I was pretty sure I had seen the error message before, but couldn't figure out where. I chased this down a bit. It looks like what is happening is that one of the (random) state space systems that is getting generated in the unit test has a near pole-zero cancellation. Here are the systems that are being compared:
The cancellation is occurring in the first system but not the second. In particular, in the second system there is a zero at -2.1991448 and a pair of poles at -2.1991448 +/- 2.29466803e-07j (one of which gets apparently gets cancelled in the first system). There are two issues here:
Not sure yet what the right fix is (if any). |
Fixed in #220. |
Uh oh!
There was an error while loading. Please reload this page.
When installing
slycot
from conda-forge (v0.3.3), I get the following error when runningpython setup.py test
:This error does not occur when using pip version of
slycot
. Need to understand what is going on before releasing 0.8.0.Note: conda-forge version of
slycot
uses OpenBLAS, pip version useslapack
(I think).The text was updated successfully, but these errors were encountered: