Skip to content
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

Revert openblas #13

Merged
merged 1 commit into from
Sep 15, 2020
Merged

Revert openblas #13

merged 1 commit into from
Sep 15, 2020

Conversation

rodluger
Copy link
Collaborator

@fbartolic fbartolic merged commit 359620c into master Sep 15, 2020
@fbartolic
Copy link
Owner

@rodluger I'm getting the following error after this merge:

WARNING (theano.tensor.blas): We did not find a dynamic library in the library_dir of the library we use for blas. If you use ATLAS, make sure to compile it with dynamics library.

This slows everything down by at least an order of magnitude.

I tried explicitly pointing theano to the library by adding export THEANO_FLAGS=blas.ldflags="-L/usr/lib/ -lopenblas" to azure-pipelines.yml as suggested here but it didn't work. Did you have this problem before?

@rodluger
Copy link
Collaborator Author

Looks like you were getting this warning before the merge as well:
https://dev.azure.com/constan7in3/volcano/_build/results?buildId=53&view=logs&j=d0f5d444-fce9-5962-ba7d-d71684238647&t=381d5e61-fc54-5591-7254-12d08113255d&l=18

Looking at my starry builds, it seems that I've always had this warning on Azure, but it's never noticeably impacted performance for me.

So, it doesn't seem like this is the issue causing the slowdown...

@fbartolic
Copy link
Owner

Hmmm, there's definitely a noticeable slowdown compared to some previous builds. I might just give up on trying to build figures on azure again.

@dfm
Copy link
Collaborator

dfm commented Sep 18, 2020

I don't understand why you're installing openblas anyways? MKL already implements everything you need and it should be better.

fbartolic added a commit that referenced this pull request Mar 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants