-
Notifications
You must be signed in to change notification settings - Fork 903
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
Add support for Python 3.11, require NumPy 1.23+ #15111
Changes from 20 commits
bf0de6c
29b8b63
0b75781
d89ae6d
f10f2ba
971ee7a
e2a6bda
e298c3a
37c675a
a8fd0d5
7310cd7
cc4d677
e299cea
72e8e29
1815ddc
7ac7fec
c1d0639
ba5e9dd
43b1283
f3f7d82
351890d
7fb5d39
a434cbd
016d1aa
6d4f1a9
2f0a8a8
631e13b
0ca1d9b
d7231a0
c36707a
dd013bd
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -92,7 +92,7 @@ jobs: | |
secrets: inherit | ||
uses: rapidsai/shared-workflows/.github/workflows/[email protected] | ||
with: | ||
matrix_filter: map(select(.ARCH == "amd64" and .PY_VER == "3.10" and (.CUDA_VER == "11.8.0" or .CUDA_VER == "12.2.2"))) | ||
matrix_filter: map(select(.ARCH == "amd64" and .PY_VER == "3.11" and (.CUDA_VER == "11.8.0" or .CUDA_VER == "12.2.2"))) | ||
build_type: ${{ inputs.build_type || 'branch' }} | ||
branch: ${{ inputs.branch }} | ||
sha: ${{ inputs.sha }} | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -128,15 +128,15 @@ jobs: | |
secrets: inherit | ||
uses: rapidsai/shared-workflows/.github/workflows/[email protected] | ||
with: | ||
matrix_filter: map(select(.ARCH == "amd64" and .PY_VER == "3.10" and (.CUDA_VER == "11.8.0" or .CUDA_VER == "12.2.2"))) | ||
matrix_filter: map(select(.ARCH == "amd64" and .PY_VER == "3.11" and (.CUDA_VER == "11.8.0" or .CUDA_VER == "12.2.2"))) | ||
build_type: pull-request | ||
script: "ci/build_wheel_dask_cudf.sh" | ||
wheel-tests-dask-cudf: | ||
needs: wheel-build-dask-cudf | ||
secrets: inherit | ||
uses: rapidsai/shared-workflows/.github/workflows/[email protected] | ||
with: | ||
matrix_filter: map(select(.ARCH == "amd64" and .PY_VER == "3.10" and (.CUDA_VER == "11.8.0" or .CUDA_VER == "12.2.2"))) | ||
matrix_filter: map(select(.ARCH == "amd64" and .PY_VER == "3.11" and (.CUDA_VER == "11.8.0" or .CUDA_VER == "12.2.2"))) | ||
build_type: pull-request | ||
script: ci/test_wheel_dask_cudf.sh | ||
devcontainer: | ||
|
@@ -152,7 +152,7 @@ jobs: | |
secrets: inherit | ||
uses: rapidsai/shared-workflows/.github/workflows/[email protected] | ||
with: | ||
matrix_filter: map(select(.ARCH == "amd64" and .PY_VER == "3.10" and (.CUDA_VER == "11.8.0" or .CUDA_VER == "12.2.2"))) | ||
matrix_filter: map(select(.ARCH == "amd64" and .PY_VER == "3.11" and (.CUDA_VER == "11.8.0" or .CUDA_VER == "12.2.2"))) | ||
build_type: pull-request | ||
script: ci/cudf_pandas_scripts/run_tests.sh | ||
# pandas-tests: | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -99,7 +99,7 @@ jobs: | |
secrets: inherit | ||
uses: rapidsai/shared-workflows/.github/workflows/[email protected] | ||
with: | ||
matrix_filter: map(select(.ARCH == "amd64" and .PY_VER == "3.10" and (.CUDA_VER == "11.8.0" or .CUDA_VER == "12.2.2"))) | ||
matrix_filter: map(select(.ARCH == "amd64" and .PY_VER == "3.11" and (.CUDA_VER == "11.8.0" or .CUDA_VER == "12.2.2"))) | ||
build_type: nightly | ||
branch: ${{ inputs.branch }} | ||
date: ${{ inputs.date }} | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -65,6 +65,7 @@ requirements: | |
- scikit-build-core >=0.7.0 | ||
- setuptools | ||
- dlpack >=0.5,<0.6.0a0 | ||
- numpy 1.23 | ||
- pyarrow ==14.0.2.* | ||
- libcudf ={{ version }} | ||
- rmm ={{ minor_version }} | ||
|
@@ -83,7 +84,7 @@ requirements: | |
- pandas >=2.0,<2.2.2dev0 | ||
- cupy >=12.0.0 | ||
- numba >=0.57 | ||
- numpy >=1.21 | ||
- {{ pin_compatible('numpy', max_pin='x') }} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Switch to using |
||
- {{ pin_compatible('pyarrow', max_pin='x') }} | ||
- libcudf ={{ version }} | ||
- {{ pin_compatible('rmm', max_pin='x.x') }} | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -258,13 +258,17 @@ dependencies: | |
- *cmake_ver | ||
- cython>=3.0.3 | ||
- *ninja | ||
- &numpy numpy>=1.21 | ||
# Hard pin the patch version used during the build. This must be kept | ||
# in sync with the version pinned in get_arrow.cmake. | ||
- pyarrow==14.0.2.* | ||
- output_types: conda | ||
packages: | ||
- scikit-build-core>=0.7.0 | ||
- output_types: pyproject | ||
packages: | ||
# Hard pin the patch version used during the build. | ||
# Sync with conda build constraint & wheel run constraint. | ||
- numpy==1.23.* | ||
Comment on lines
+266
to
+270
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is the same NumPy pin the conda packages use to build. It is now used here to align the wheel packages |
||
- output_types: [requirements, pyproject] | ||
packages: | ||
- scikit-build-core[pyproject]>=0.7.0 | ||
|
@@ -488,15 +492,19 @@ dependencies: | |
py: "3.10" | ||
packages: | ||
- python=3.10 | ||
- matrix: | ||
py: "3.11" | ||
packages: | ||
- python=3.11 | ||
- matrix: | ||
packages: | ||
- python>=3.9,<3.11 | ||
- python>=3.9,<3.12 | ||
run_common: | ||
common: | ||
- output_types: [conda, requirements, pyproject] | ||
packages: | ||
- fsspec>=0.6.0 | ||
- *numpy | ||
- numpy>=1.23 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Both conda & wheel packages now have the same lower bound on NumPy |
||
- pandas>=2.0,<2.2.2dev0 | ||
run_cudf: | ||
common: | ||
|
@@ -625,7 +633,7 @@ dependencies: | |
- output_types: pyproject | ||
packages: | ||
- msgpack | ||
- &tokenizers tokenizers==0.13.1 | ||
- &tokenizers tokenizers==0.15.2 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I've bumped tokenizers to 0.15.2 so we can get Python 3.11 binaries. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Incompatible with There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Bumped transformers to 4.38.1. |
||
- &transformers transformers==4.24.0 | ||
- tzdata | ||
specific: | ||
|
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.
As NumPy is used at build time, this now adds it to
requirements/host