Skip to content
This repository has been archived by the owner on Sep 29, 2022. It is now read-only.

nomkl: Add recipe for building metapackage. #586

Closed
wants to merge 1 commit into from

Conversation

jakirkham
Copy link
Contributor

Simply adds the nomkl package. Seems like some people didn't have this one Windows. ( conda/conda#2032 (comment) ) So, this gives them a way to build it. This, at least, seems to be how it works, but I could be missing something. Feedback welcome.

@msarahan
Copy link
Collaborator

I think this is how it works, but the real reason it doesn't exist on windows is that we don't build non-mkl packages of numpy, scipy and friends. Those are the packages that get picked up by this feature. For this package to be useful, there also need to be openblas (or some alternative blas) linked versions, and those need to provide the nomkl feature.

@msarahan
Copy link
Collaborator

The old numpy/scipy builds were repackaged Gohlke packages. These also had
mkl. We are pushing towards building everything from source ourselves. At
present, there are no non-mkl builds on windows.

On Wed, Feb 10, 2016, 07:56 jakirkham notifications@github.com wrote:

On Windows, aren't these supplied anyways or are those packages missing
too?


Reply to this email directly or view it on GitHub
#586 (comment).

@jakirkham
Copy link
Contributor Author

Ah, ok, I was always curious what BLAS was being used for NumPy and SciPy on Windows. It looked like none was included, but if that were the case performance would suffer. Thanks for explaining what was going on.

Well having an OpenBLAS build could be good. Plus more recent versions of OpenBLAS support CMake for the very reason of having a usable Windows build system. Maybe it would be worth giving that a shot. If I had a working dev environment for Windows, I would give it a shot. At present, I don't. Though it looks like this should be straightforward. ( https://github.com/xianyi/OpenBLAS/blob/develop/appveyor.yml#L38-L39 )

@msarahan
Copy link
Collaborator

That does look pretty straightforward. I'll add it to our todo-list. No promise on when it will happen, though. =(

@jakirkham
Copy link
Contributor Author

Yeah, I know, you guys are swamped.

As a side note, it appears this will use the C kernels on Windows, not the assembly kernels, as the assembly code style is not supported by MSVC. Not sure when or if that will change.

@jakirkham
Copy link
Contributor Author

Probably will still want this though as the openblas based recipes now need nomkl features added as well and it would be wise to have this here just in case.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants