-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Error in single threaded positive definite inverse in OpenBLAS #126
Comments
the linalg tests are already really long, but could any additional tests have caught the difference between single and multi threaded openblas here? |
I think, it only happened for large matrices, so we can just set |
It seems like the linalg tests are becoming mostly a unit test suite for OpenBLAS. |
Yeah, largely. Their own tests are more comprehensive than ours, but Julia gets installed on a wider variety of machines than the OpenBLAS devs have access to. |
Maybe we should have a small set of tests that are run with different problem sizes and with different numbers of threads. |
OpenBLAS 0.2.10 is out with the fix for this, time for another bump? |
Could you test it out and bump it? I am just boarding a flight to India and will be mostly in an aluminium tube for the next 30 hours or so. |
Hm, judging by OpenMathLib/OpenBLAS#415 there may be trouble on 32 bit Linux? |
Looks like it has been fixed and there is a new 0.2.10 tag. |
I'm currently testing out 0.2.10 on Ubuntu and OSX. I will submit a PR when I've finished. |
@staticfloat Did this work out? |
Through my adventures, we've gotten a re-tagged 0.2.10 (along with a promise from the devs that they'll try to just release new versions instead of retagging). PR is here, but it would be nice if someone could verify that the issue reported in this thread is actually solved by version bump. |
I just merged it, thinking from the comments in the PR that it was good to go. Anyways, I am building right now and will speak up if it fails. |
I reproduced this problem in a pre-0.2.10 version and saw it fixed in the first tagged version of 0.2.10, which differed only in a single line from the second tag, so I think we're okay |
I don't see the PosDef exception any more. |
I wonder if we can print a line in the banner encouraging everyone who downloads julia to run the tests during the RC stage and report issues. |
Why just during the RC stage? Should put it on the download page. Julia is under heavy development, here's how to run the test suite first thing after installing a binary, if it doesn't pass then search github for similar issues and add a comment if you see something that's already been reported, or open a new issue if you can't find anything similar. |
That is a valid point, and I actually like the idea. The downloads page can have detailed instructions, and the banner can have something just as a reminder. Perhaps the request to run tests can be made quiet after the tests are run for that particular version. |
Maybe a |
Great idea! Perhaps we should have a new issue for this. |
As reported at the list
Issue filed: OpenMathLib/OpenBLAS#410
The text was updated successfully, but these errors were encountered: