-
Notifications
You must be signed in to change notification settings - Fork 5.4k
Use OpenBLAS 0.3.7 that allows USE_LOCKING flag for better multithrea… #3538
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
Conversation
|
That sounds good-- those crashes have been a nasty headache. Would you mind just running the matrix speed test and make sure there isn't a major change in speed? Hopefully it would only acquire locks for larger operations. |
|
Sorry for the late reply, I will try testing and report back as soon as I can. |
|
@danpovey Ok, I compiled 0.3.7 without and with Without With Consecutive calls are yielding slightly different results. But seems ok to me. |
|
@ahmetaa can you please recreate this using a branch name that is not 'master' and make sure any conflicts are addressed? I am scared to merge this because I think my attempt to resolve a conflict via the web interface has somehow confused GitHub and I don't know exactly how it will behave if I merge it now. |
|
@danpovey Sure, will do soon. |
…ding support.