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

Dask-CUDA: CUDA 12 Conda Packages #1115

Closed
10 tasks done
jakirkham opened this issue Feb 14, 2023 · 6 comments · Fixed by #1201
Closed
10 tasks done

Dask-CUDA: CUDA 12 Conda Packages #1115

jakirkham opened this issue Feb 14, 2023 · 6 comments · Fixed by #1201
Assignees

Comments

@jakirkham
Copy link
Member

jakirkham commented Feb 14, 2023

Pushing for CUDA 12 Conda package builds of libraries for RAPIDS 23.04. This issue is for tracking that work.

Requirements (please recheck):

Developer tasks

Dependencies from Conda

RAPIDS Conda dependencies

  1. 6 of 6
    bdice
  2. 15 of 15
    bdice
  3. feature request non-breaking
    bdice
  4. 7 of 7
    pentschev

Optional dependencies from Conda

@wence-
Copy link
Contributor

wence- commented Feb 14, 2023

(via distributed/ucx-py)

Not sure what this is supposed to mean. Do you just want to write ucx-py (RAPIDS) ?

Also not sure how you're specifying transitive dependencies, e.g. if you're optionally depending on cudf, do you also need to depend on rmm (which is a transitive dep of cudf)?

@jakirkham
Copy link
Member Author

^ @pentschev, do you have thoughts on this question?

@pentschev
Copy link
Member

(via distributed/ucx-py)

Not sure what this is supposed to mean. Do you just want to write ucx-py (RAPIDS) ?

I just tried to be somewhat verbose, not even sure this is a necessity. What I meant to say is that the dependency is Dask-CUDA->Distributed->UCX-Py->UCX, and it is an important dependency for us, thus the verbosity. I'll defer to @jakirkham on how that can be best viewed.

Also not sure how you're specifying transitive dependencies, e.g. if you're optionally depending on cudf, do you also need to depend on rmm (which is a transitive dep of cudf)?

Again just for verbosity. Users may have cuDF workflows that depend on RMM, or just CuPy (for example) workflows that also depend on RMM. With that said, I just wanted to make sure this is properly visualized, if this is only overhead I'm fine reorganizing or removing transitive dependencies altogether, probably @jakirkham has a best idea of what's important and could remove any overheads.

@jakirkham
Copy link
Member Author

Yeah no strong feelings here. Understand what is written above. Ok with the other alternatives mentioned in the thread

@jakirkham
Copy link
Member Author

In PR ( #1130 ), we are trying to update to PyNVML 11.5.0, but are having some issues ( #1130 (comment) )

AIUI PyNVML 11.5.0 is compatible with CUDA 12. Checking to see if an older PyNVML version would suffice

@jakirkham
Copy link
Member Author

After checking offline, have confirmed we need PyNVML 11.5.0+ for CUDA 12 support.

@jakirkham jakirkham changed the title CUDA 12 Conda Packages Dask-CUDA: CUDA 12 Conda Packages Apr 7, 2023
rapids-bot bot pushed a commit that referenced this issue Jun 28, 2023
Adds CUDA 12 Support to build matrix

Fixes #1115

Authors:
  - Benjamin Zaitlen (https://github.com/quasiben)
  - Peter Andreas Entschev (https://github.com/pentschev)

Approvers:
  - Bradley Dice (https://github.com/bdice)
  - Ray Douglass (https://github.com/raydouglass)
  - https://github.com/jakirkham

URL: #1201
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 a pull request may close this issue.

4 participants