Skip to content

Commit

Permalink
WIP: put a ceiling on cuda-python (#1723)
Browse files Browse the repository at this point in the history
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
  • Loading branch information
jameslamb authored Nov 7, 2024
1 parent d4c0635 commit a98c22a
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 10 deletions.
2 changes: 1 addition & 1 deletion conda/environments/all_cuda-118_arch-x86_64.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ dependencies:
- clang-tools==16.0.6
- clang==16.0.6
- cmake>=3.26.4,!=3.30.0
- cuda-python>=11.7.1,<12.0a0,!=11.8.4
- cuda-python>=11.7.1,<12.0a0,<=11.8.3
- cuda-version=11.8
- cudatoolkit
- cxx-compiler
Expand Down
2 changes: 1 addition & 1 deletion conda/environments/all_cuda-125_arch-x86_64.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ dependencies:
- clang==16.0.6
- cmake>=3.26.4,!=3.30.0
- cuda-nvcc
- cuda-python>=12.0,<13.0a0,!=12.6.1
- cuda-python>=12.0,<13.0a0,<=12.6.0
- cuda-version=12.5
- cxx-compiler
- cython>=3.0.0
Expand Down
8 changes: 4 additions & 4 deletions conda/recipes/rmm/meta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -57,10 +57,10 @@ requirements:
- cuda-version ={{ cuda_version }}
{% if cuda_major == "11" %}
- cudatoolkit
- cuda-python >=11.7.1,<12.0a0,!=11.8.4
- cuda-python >=11.7.1,<12.0a0,<=11.8.3
{% else %}
- cuda-cudart-dev
- cuda-python >=12.0,<13.0a0,!=12.6.1
- cuda-python >=12.0,<13.0a0,<=12.6.0
{% endif %}
- cython >=3.0.0
- rapids-build-backend >=0.3.0,<0.4.0.dev0
Expand All @@ -70,10 +70,10 @@ requirements:
run:
{% if cuda_major == "11" %}
- cudatoolkit
- cuda-python >=11.7.1,<12.0a0,!=11.8.4
- cuda-python >=11.7.1,<12.0a0,<=11.8.3
{% else %}
- cuda-cudart
- cuda-python >=12.0,<13.0a0,!=12.6.1
- cuda-python >=12.0,<13.0a0,<=12.6.0
{% endif %}
- {{ pin_compatible('cuda-version', max_pin='x', min_pin='x') }}
- numba >=0.57
Expand Down
4 changes: 2 additions & 2 deletions dependencies.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -153,10 +153,10 @@ dependencies:
- matrix:
cuda: "12.*"
packages:
- &cuda_python12 cuda-python>=12.0,<13.0a0,!=12.6.1
- &cuda_python12 cuda-python>=12.0,<13.0a0,<=12.6.0
- matrix: # All CUDA 11 versions
packages:
- &cuda_python11 cuda-python>=11.7.1,<12.0a0,!=11.8.4
- &cuda_python11 cuda-python>=11.7.1,<12.0a0,<=11.8.3
- output_types: [requirements, pyproject]
matrices:
- matrix:
Expand Down
4 changes: 2 additions & 2 deletions python/rmm/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ authors = [
license = { text = "Apache 2.0" }
requires-python = ">=3.10"
dependencies = [
"cuda-python>=11.7.1,<12.0a0,!=11.8.4",
"cuda-python>=11.7.1,<12.0a0,<=11.8.3",
"numba>=0.57",
"numpy>=1.23,<3.0a0",
] # This list was generated by `rapids-dependency-file-generator`. To make changes, edit ../../dependencies.yaml and run `rapids-dependency-file-generator`.
Expand Down Expand Up @@ -128,7 +128,7 @@ dependencies-file = "../../dependencies.yaml"
matrix-entry = "cuda_suffixed=true"
requires = [
"cmake>=3.26.4,!=3.30.0",
"cuda-python>=11.7.1,<12.0a0,!=11.8.4",
"cuda-python>=11.7.1,<12.0a0,<=11.8.3",
"cython>=3.0.0",
"librmm==24.12.*,>=0.0.0a0",
"ninja",
Expand Down

0 comments on commit a98c22a

Please sign in to comment.