-
Notifications
You must be signed in to change notification settings - Fork 21.4k
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
Move redundant code that checks NumPy during build to a helper module and add an option to disable building with NumPy #21417
Conversation
@soumith I've pushed another commit that adds an option for enabling and disabling NumPy build. Could you review again? If you feel it's too hard to review, I can separate it into a different PR after this is merged. Thanks! |
7d23a56
to
2399621
Compare
4670d08
to
aaa1dc3
Compare
aaa1dc3
to
f41d18d
Compare
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.
@fmassa has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
@fmassa go ahead and merge |
… and add an option to disable building with NumPy Summary: Pull Request resolved: pytorch/pytorch#21417 Reviewed By: ezyang Differential Revision: D15694357 Pulled By: fmassa fbshipit-source-id: bc1bda23349ba4531f19619fa4adecb846225c20
No description provided.