Skip to content
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

Use conda strict channel priority. #109

Merged
merged 1 commit into from
Oct 24, 2024
Merged

Conversation

bdice
Copy link
Contributor

@bdice bdice commented Oct 24, 2024

This attempts to resolve some issues with CUDA packages being pulled from both conda-forge and nvidia channels by using strict channel priority.

The nvidia channel should not be needed for this package.

@bdice bdice mentioned this pull request Oct 24, 2024
@bdice bdice marked this pull request as ready for review October 24, 2024 18:31
@bdice bdice requested a review from a team as a code owner October 24, 2024 18:31
@bdice bdice requested a review from jameslamb October 24, 2024 18:31
@bdice
Copy link
Contributor Author

bdice commented Oct 24, 2024

@gmarkall @jakirkham @jameslamb Using strict channel priority unblocks CI. I don't know what the exact root cause is, but it's something with nvidia / conda-forge channel mixing as @jakirkham noted here: #107 (comment)

I propose we merge this PR, close #107, merge #106, then merge #97.

@bdice bdice merged commit a823705 into rapidsai:main Oct 24, 2024
39 checks passed
@jakirkham
Copy link
Member

Excellent! Thanks all 🙏

@jakirkham jakirkham mentioned this pull request Oct 24, 2024
gmarkall pushed a commit that referenced this pull request Oct 25, 2024
* Fix building tests in multi-gpu environment (#98) 
* Change cmake.verbose = true to build.verbose = true (#99) 
* Use build-system.requires to set scikit-build-core minimum version
(#100)
* Set CUDA version in one file (and use everywhere else) (#101) 
* Drop Python 3.9 support (#102) 
* Use CI workflow branch 'branch-24.10' again (#105) 
* Use conda strict channel priority. (#109) 
* Update to CUDA 12.6 (#97)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants