-
Notifications
You must be signed in to change notification settings - Fork 446
Unpin numpy from <2.0 #2459
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
Unpin numpy from <2.0 #2459
Conversation
This was done in meta-pytorch#2382 to avoid issues with windows builds.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2459 +/- ##
=======================================
Coverage 99.98% 99.98%
=======================================
Files 191 191
Lines 16795 16795
=======================================
Hits 16793 16793
Misses 2 2 ☔ View full report in Codecov by Sentry. |
I guess this still causes issues, will need to dig deeper. |
In CI we were using `pip install torch torchvision -f https://download.pytorch.org/whl/nightly/cpu/torch_nightly.html` whereas the pytorch website sugggests `pip install --pre torch torchvision --index-url https://download.pytorch.org/whl/nightly/cpu`. Hopefully this will pick up newer nightly builds and resolve the windows numpy compatgibility issue.
The issue appears to be that the windows binaries for pytorch 2.4.0 are not compatible with pytorch/builder#1945 should fix this going forward for the nightly builds in our CI, but fundamentally this isn't actually a botorch issue but a torch/numpy compatibility issue on windows. I also realized that in our CI we were using |
This appears to be the case. There is still the issue of existing torch release windows binaries potentially not being compatible with numpy 2.0, but that shouldn't be something that we need to handle. |
@Balandat has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
1 similar comment
@Balandat has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
This unpins numpy from <2.0, which was done as a temporary workaround in #2382 to avoid issues with windows builds.
The underlying issue was an incompatibility between pytorch 2.4.0 windows builds and numpy 2.0. Pointed pip to the updated pytorch pip index to pick up newer 2.5.0 dev versions to resolve the issue: #2459 (comment)
There is still the issue of existing torch release windows binaries potentially not being compatible with numpy 2.0, but that shouldn't be something that we need to handle.