-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
493d281
commit 2edbe6c
Showing
1 changed file
with
1 addition
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2edbe6c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ColCarroll Why is mkl-service needed in PyMC3? Is this Intel MKL library support or something else?
2edbe6c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mostly addressed the reasoning in the PR for this (#1572). It is Intel MKL optimizations. I'm not super familiar with the differences in optimization libraries, but theano was issuing a warning if it wasn't installed. If you have some expertise, would love to learn!
Note that this does not install mkl-service when pip installing or conda installing
pymc3
.2edbe6c
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ColCarroll Thanks for the details! I checked my win-64 anaconda install and noticed that "mkl-service" package has been already installed as a dependency to some of my packages of interest. PyMC3 is installed in the same environment; it must be other python package then which pulled it as a dependency.