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.

Sign up for GitHub

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

Jump to bottom

Disallow cuda-python 12.6.1 / 11.8.4 #116

Closed
bdice opened this issue Nov 6, 2024 · 1 comment
Closed

Disallow cuda-python 12.6.1 / 11.8.4 #116

bdice opened this issue Nov 6, 2024 · 1 comment
Assignees
@bdice @jameslamb

Comments

@bdice
Copy link
Contributor

bdice commented Nov 6, 2024

Due to a bug in cuda-python we must disallow cuda-python 12.6.1 and 11.8.4. NVIDIA/cuda-python#215

We expect 12.6.2 and 11.8.5 to correct this bug. I will file PRs to all of RAPIDS to pin cuda-python !=12.6.1 / !=11.8.4.
@bdice bdice self-assigned this Nov 6, 2024
This was referenced Nov 6, 2024
Merged
Merged
Merged
Merged
Merged
Merged
Merged
@jameslamb jameslamb self-assigned this Nov 6, 2024
rapids-bot bot pushed a commit to rapidsai/kvikio that referenced this issue Nov 6, 2024
@bdice
Disallow cuda-python 12.6.1 and 11.8.4 (#537)
dac0803 
Due to a bug in cuda-python we must disallow cuda-python 12.6.1 and 11.8.4. This PR disallows those versions.

See rapidsai/build-planning#116 for more information.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - https://github.com/jakirkham

URL: #537
rapids-bot bot pushed a commit to rapidsai/rmm that referenced this issue Nov 6, 2024
@bdice
Disallow cuda-python 12.6.1 and 11.8.4 (#1720)
d4c0635 
Due to a bug in cuda-python we must disallow cuda-python 12.6.1 and 11.8.4. See rapidsai/build-planning#116 for more information.

This PR disallows those versions, and other changes following from that:

* specifying `python` in both `host:` and `run:` dependencies for the `rmm` conda package
* ignoring deprecation warnings raised by newer versions of `cuda-python`

Authors:
  - Bradley Dice (https://github.com/bdice)
  - James Lamb (https://github.com/jameslamb)
  - https://github.com/jakirkham

Approvers:
  - James Lamb (https://github.com/jameslamb)

URL: #1720
@vyasr vyasr mentioned this issue Nov 6, 2024
Open
AyodeAwe pushed a commit to rapidsai/cudf that referenced this issue Nov 6, 2024
@bdice @jameslamb
Disallow cuda-python 12.6.1 and 11.8.4 (#17253)
06b3f83 
Due to a bug in cuda-python we must disallow cuda-python 12.6.1 and
11.8.4. This PR disallows those versions. It also silences new
cuda-python deprecation warnings so that our test suite passes.

See rapidsai/build-planning#116 for more
information.

---------

Co-authored-by: James Lamb <[email protected]>
This was referenced Nov 7, 2024
Merged
Merged
Merged
rapids-bot bot pushed a commit to rapidsai/rmm that referenced this issue Nov 7, 2024
@jameslamb
WIP: put a ceiling on cuda-python (#1723)
a98c22a 
Follow-up to #1720

Contributes to rapidsai/build-planning#116

That PR used `!=` requirements to skip a particular version of `cuda-python` that `rmm` was incompatible with. A newer version of `cuda-python` (12.6.2 for CUDA 12, 11.8.5 for CUDA 11) was just released, and it also causes some build issues for RAPIDS libraries: rapidsai/cuvs#445 (comment)

To unblock CI across RAPIDS, this proposes **temporarily** switching to ceilings on `rmm`'s `cuda-python` dependency.

Authors:
  - James Lamb (https://github.com/jameslamb)

Approvers:
  - Vyas Ramasubramani (https://github.com/vyasr)

URL: #1723
rapids-bot bot pushed a commit to rapidsai/kvikio that referenced this issue Nov 7, 2024
@jameslamb
Put a ceiling on cuda-python (#543)
b6a2655 
Follow-up to #537

Contributes to rapidsai/build-planning#116

That PR used `!=` requirements to skip a particular version of `cuda-python` that `kvikio` was incompatible with. A newer version of `cuda-python` (12.6.2 for CUDA 12, 11.8.5 for CUDA 11) was just released, and it also causes some build issues for RAPIDS libraries: rapidsai/cuvs#445 (comment)

To unblock CI across RAPIDS, this proposes **temporarily** switching to ceilings on the `cuda-python` dependency here.

Authors:
  - James Lamb (https://github.com/jameslamb)

Approvers:
  - Vyas Ramasubramani (https://github.com/vyasr)

URL: #543
rapids-bot bot pushed a commit to rapidsai/cudf that referenced this issue Nov 7, 2024
@jameslamb
Put a ceiling on cuda-python (#17264)
29484cb 
Follow-up to #17253

Contributes to rapidsai/build-planning#116

That PR used `!=` requirements to skip a particular version of `cuda-python` that `cudf` and `pylibcudf` were incompatible with. A newer version of `cuda-python` (12.6.2 for CUDA 12, 11.8.5 for CUDA 11) was just released, and it also causes some build issues for RAPIDS libraries: rapidsai/cuvs#445 (comment)

To unblock CI across RAPIDS, this proposes **temporarily** switching to ceilings on the `cuda-python` dependency here.

Authors:
  - James Lamb (https://github.com/jameslamb)

Approvers:
  - Vyas Ramasubramani (https://github.com/vyasr)
  - Bradley Dice (https://github.com/bdice)

URL: #17264
rapids-bot bot pushed a commit to rapidsai/raft that referenced this issue Nov 7, 2024
@bdice
Put a ceiling on cuda-python (#2486)
615447f 
This project is incompatible with newer versions of `cuda-python`. This puts ceilings of `<=11.8.3` (CUDA 11) and `<=12.6.0` (CUDA 12) on that library.

Those ceilings should be removed and replaced with `!=` constraints once new releases of `cuda-python` are up that this project is compatible with.

See rapidsai/build-planning#116 for more information.

Authors:
  - Bradley Dice (https://github.com/bdice)
  - James Lamb (https://github.com/jameslamb)

Approvers:
  - Vyas Ramasubramani (https://github.com/vyasr)

URL: #2486
rapids-bot bot pushed a commit to rapidsai/cuvs that referenced this issue Nov 7, 2024
@bdice
Put a ceiling on cuda-python (#445)
2d4afb5 
This project is incompatible with newer versions of `cuda-python`. This puts ceilings of `<=11.8.3` (CUDA 11) and `<=12.6.0` (CUDA 12) on that library.

Those ceilings should be removed and replaced with `!=` constraints once new releases of `cuda-python` are up that this project is compatible with.

See rapidsai/build-planning#116 for more information.

Authors:
  - Bradley Dice (https://github.com/bdice)
  - James Lamb (https://github.com/jameslamb)

Approvers:
  - James Lamb (https://github.com/jameslamb)
  - Divye Gala (https://github.com/divyegala)

URL: #445
rapids-bot bot pushed a commit to rapidsai/cugraph that referenced this issue Nov 7, 2024
@bdice
Put a ceiling on cuda-python (#4747)
0e57e65 
This project is incompatible with newer versions of `cuda-python`. This puts ceilings of `<=11.8.3` (CUDA 11) and `<=12.6.0` (CUDA 12) on that library.

Those ceilings should be removed and replaced with `!=` constraints once new releases of `cuda-python` are up that this project is compatible with.

See rapidsai/build-planning#116 for more information.

Authors:
  - Bradley Dice (https://github.com/bdice)
  - James Lamb (https://github.com/jameslamb)

Approvers:
  - James Lamb (https://github.com/jameslamb)

URL: #4747
rapids-bot bot pushed a commit to rapidsai/cuml that referenced this issue Nov 7, 2024
@bdice
Put a ceiling on cuda-python (#6131)
9150279 
This project is incompatible with newer versions of `cuda-python`. This puts ceilings of `<=11.8.3` (CUDA 11) and `<=12.6.0` (CUDA 12) on that library.

Those ceilings should be removed and replaced with `!=` constraints once new releases of `cuda-python` are up that this project is compatible with.

See rapidsai/build-planning#116 for more information.

Authors:
  - Bradley Dice (https://github.com/bdice)
  - James Lamb (https://github.com/jameslamb)

Approvers:
  - James Lamb (https://github.com/jameslamb)

URL: #6131
@jameslamb
Copy link
Member

All of the PRs updating pins to skip these versions have been merged, and this issue is no longer affecting RAPIDS CI.

The only remaining open item is the rapids-reviser PR (https://github.com/rapidsai/rapids-reviser/pull/56). That isn't a critical part of this, so I'm closing this issue in the interest of narrowing down the list of what's left to be done for 24.12.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bdice bdice

@jameslamb jameslamb

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bdice @jameslamb