-
Notifications
You must be signed in to change notification settings - Fork 902
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
Ensure dev environment uses Arrow GPU packages #8637
Ensure dev environment uses Arrow GPU packages #8637
Conversation
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.
Do you know if the formatting for the recipe is correct? Not sure if the syntax differs significantly from the conda env file |
Codecov Report
@@ Coverage Diff @@
## branch-21.08 #8637 +/- ##
===============================================
Coverage ? 10.61%
===============================================
Files ? 109
Lines ? 18648
Branches ? 0
===============================================
Hits ? 1980
Misses ? 16668
Partials ? 0 Continue to review full report at Codecov.
|
Co-authored-by: jakirkham <[email protected]>
@gpucibot merge |
Thanks all! 😄 |
Looks like we needed another change to the recipe. Added here ( #8651 ) |
Follow up to PR ( #8637 ) Even though we pinned `*cuda` in `host`, this didn't carry over to the `run` dependencies. To fix this, tack on `*cuda` to `pyarrow` and `arrow-cpp` in `run`. This should fix installation errors where the CPU builds were getting pulled in unintentionally. cc @ajschmidt8 @charlesbluca @pentschev Authors: - https://github.com/jakirkham Approvers: - Jordan Jacobelli (https://github.com/Ethyling) URL: #8651
With #7495 merged, it seems like the dev environment files create an environment with the CPU packages for
pyarrow
andarrow-cpp
; this results in failure when trying to compile libcudf orimport cudf
:This updates the dev environments and recipe to ensure that the GPU package of
pyarrow
(andarrow-cpp
accordingly) are used.