-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Unexpected nans in complex cholfact #260
Comments
julia.mit.edu has old processors so maybe that is the reason. I don't get the |
How about on recent macs or newer processor linux? |
Just tried on my Haswell MacBook Pro and a Haswell Dell running Ubuntu 15.04 and there are no problems. |
Can you reproduce by setting
|
My Haswell based Macbook Pro gives
|
Great, thanks for running those. On a few different machines I tried I was either always getting the nan or never getting it. So Sandybridge and Haswell kernels aren't hitting the bug, but earlier ones are. |
Worked around upstream with the latest openblas release |
This is related to #245 but I can get it to happen on julia.mit.edu without setting
OPENBLAS_CORETYPE
to use old kernels, and it persists even with the latest develop branch of openblas:This results in
test failed: NaN32 <= 9.709564f-6
. This was reduced from a test failure that was happening repeatably in b512505cacc69fd1ba2cf04ecca79cee587a4cfd, which was in a WIP version of JuliaLang/julia#12870 (you can add and fetch from my fork as a remote for an easier time checking out that commit, since it is no longer present on any branch here), ref comments at https://github.com/JuliaLang/julia/blob/8681313fbab1dcd88b8a9fe9b56f91edd0b75f9b/test/linalg/cholesky.jl#L41-L43. I think this qualifies as "worth reporting upstream" but might make their lives easier to translate the test case to C or Fortran first.The text was updated successfully, but these errors were encountered: