-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DOC: improve documentation for building extensions on windows #12428
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
Closed
Closed
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
If they're going to support it in
numpy
andscipy
, I supposepandas
can do the same: I can installpandas
on Cygwin as well using itsgcc
compiler. That would be worth mentioning in case someone already hasnumpy
installed on Cygwin.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 problem with adding this at some point. but AFAICT this is a bit non-trivial ATM. There are straightforwad ways to build extensions now that have been long in practice and are fully compatible. Let's have numpy/scipy use and abuse then can build off of that. You can create another issue (with some links) and I will mark similarly.
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 don't have a link, but I just did it about ten minutes ago. All you need are the
gcc
packages and Cygwin's Python.pandas
is fully capable at this point AFAICT to allow this. In fact, it's even simpler withpandas
relative tonumpy
/scipy
because you don't need additional libraries (e.g.BLAS
) to buildpandas
.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.
@gfyoung I agree, but I would like to see this actually used (and try myself). Further AFAICT no-one distributes cygwin python. is this compat with
conda
?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 don't believe
conda
is compatible with Cygwin (at least I haven't been able to do so).I don't quite follow your comment about no one distributing Cygwin Python. It comes as an optional package during Cygwin installation.
When you say "actually used," are you referring to seeing other people using / testing with it outside of just myself?
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.
@gfyoung yes, we basically recommend using a conda install now as its the most straightforward / easiest way of doing this (on windows). my impression was that
mingw-64
just became available / stable. I'd like to see this in common use before we recommend its use to build.In any event, simply create another issue and can discuss there.
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.
Fair enough. It's not absolutely necessary to add mention of Cygwin since
conda
should work just fine in this case given the fewer constraints vs.numpy
orscipy
. I'll create another issue for it.