-
-
Notifications
You must be signed in to change notification settings - Fork 16
LAPACK error on simple SVD #325
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Comments
Ha. That's a funny one. It should be easy to work around, though. Thanks for the SciPy people. |
Changing places like https://github.com/JuliaLang/julia/blob/master/base/linalg/lapack.jl#L302 to lwork = BlasInt(nextfloat(real(work[1]))) should be enough, right? |
Almost. I think this would give an inexact error for smaller values so something like lwork = round(BlasInt, nextfloat(real(work[1]))) should work. It would be great if you could make a PR. |
sgesdd by using nextfloat. See http://icl.cs.utk.edu/lapack-forum/viewtopic.php?f=13&t=4587&p=11036&hilit=sgesdd#p11036 and scipy/scipy#5401 Fixes #15784
sgesdd by using nextfloat. Now with comments and test so this commit supersedes ad59ceb See http://icl.cs.utk.edu/lapack-forum/viewtopic.php?f=13&t=4587&p=11036&hilit=sgesdd#p11036 and scipy/scipy#5401 Fixes #15784
sgesdd by using nextfloat. Now with comments and test so this commit supersedes ad59ceb See http://icl.cs.utk.edu/lapack-forum/viewtopic.php?f=13&t=4587&p=11036&hilit=sgesdd#p11036 and scipy/scipy#5401 Fixes #15784
sgesdd by using nextfloat. Now with comments and test so this commit supersedes ad59ceb See http://icl.cs.utk.edu/lapack-forum/viewtopic.php?f=13&t=4587&p=11036&hilit=sgesdd#p11036 and scipy/scipy#5401 Fixes #15784
Just tested it with
My julia version is
|
It is fixed on master but it hasn't been backported yet. I forgot to label the fix for backporting but have done that now so it should be included in next bugfix release of 0.4. |
Great! Thanks. |
sgesdd by using nextfloat. Now with comments and test so this commit supersedes ad59ceb See http://icl.cs.utk.edu/lapack-forum/viewtopic.php?f=13&t=4587&p=11036&hilit=sgesdd#p11036 and scipy/scipy#5401 Fixes #15784 (cherry picked from commit b6988df) ref #15824
throws an error:
This sounds very close to scipy/scipy#5401, maybe we need a similar workaround?
This is on julia 0.4.5 built from source on Ubuntu 14.04.
The text was updated successfully, but these errors were encountered: