-
-
Notifications
You must be signed in to change notification settings - Fork 117
DOC: Revamp Building from Source on Windows #146
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
used to build libraries ATLAS_, which at the moment is very Unix-oriented, although | ||
that may be subject to change as we will discuss later on. If selected during installation, | ||
Cygwin_ also makes available its own versions of the MinGW_ compilers (by the command | ||
line option "-mno-cygwin" to gcc), which produce identical code. **There is no need to |
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.
this option has not worked since gcc 3 and is an error since 4.5ish
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.
Ah, good to know. Fixed.
I have a first draft of the documentation put together now in this PR. I tried to be somewhat more general compared to the old version given how much active development is going on currently. If there is anything missing or inaccurate, please let me know! |
################### | ||
|
||
OpenBLAS_ is an optimized version of BLAS that is currently used in languages like Julia_ by default. Besides being | ||
actively worked upon, it performs about as well as the Intel libraries discussed previously. Furthermore, it is also |
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.
this will install cygwin linked versions of openblas which work well with cygwin's python but probably not python.org native python
edit: this should be one line lower, sorry
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.
No worries and fair point.
@tkelman : I made your suggested changes. If there is anything else that needs to be changed or is inaccurate, do let me know! |
NumPy and SciPy both support MSVC as the C/C++ compiler extension modules for the official | ||
binary distribution of Python. However, make sure that you download the correct version! | ||
For example, Python 2.7.x is compiled with Visual Studio 2008, and Python 3.5.1 is compiled | ||
with Visual Studio 2015. Once you have found the appropriate version, visit this link `here <https://support.microsoft.com/en-gb/kb/2977003>`__ |
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.
This links to the 'Visual C++ redistributable package', which is not enough to build yourself
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.
Fixed.
Does anyone else have anything to add to this at the moment? Otherwise, it seems like it can be merged. |
also require their own Fortran compiler for these libraries to work. While it is possible to obtain | ||
the libraries for free via their Community License (you can click `here <https://software.intel.com/sites/campaigns/nest/>`__ | ||
to learn and click `here <https://registrationcenter.intel.com/en/forms/?productid=2558&licensetype=2>`__ to register), | ||
it does not come with their Fortran compiler, **ifort**, which is necessary for building both the NumPy and SciPy libraries with MKL. |
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.
So on Windows it doesn't need icc
, like it does on Linux? That's probably worth mentioning explicitly.
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.
No AFAICT. Based off the log, my builds use cl.exe
from Visual Studio. I will add a comment to make it clear that icc
is not strictly necessary.
Other than my nitpicks LGTM |
@Eric89GXL : Updated PR with your comments. If no one else has anything to add, I think this is good to merge. |
DOC: Revamp Building from Source on Windows
Thanks @gfyoung |
@Eric89GXL : Thanks! Are you able to upload the changes to the website? I still see the original documentation. |
I haven't done that before, so I'm not sure what the procedure is. |
Not to worry. Somebody already did it. I believe the command is |
Addresses issue #145 by revamping / updating the documentation on building source from Windows to provide a more organized format as well as to reflect ongoing projects like Mingwpy that users should be aware of. Related to
numpy
issue #5479@matthew-brett, @carlkl, @njsmith, @rgommers, @juliantaylor, @tkelman