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

[RELEASE] cuspatial v24.02 #1336

Merged
merged 21 commits into from
Feb 13, 2024
Merged

[RELEASE] cuspatial v24.02 #1336

merged 21 commits into from
Feb 13, 2024

Conversation

raydouglass
Copy link
Member

❄️ Code freeze for branch-24.02 and v24.02 release

What does this mean?

Only critical/hotfix level issues should be merged into branch-24.02 until release (merging of this PR).

What is the purpose of this PR?

  • Update documentation
  • Allow testing for the new release
  • Enable a means to merge branch-24.02 into main for the release

raydouglass and others added 19 commits November 9, 2023 16:32
@raydouglass
v24.02 Updates [skip ci]
71e83d9
@GPUtester
Merge pull request #1296 from rapidsai/branch-23.12
dbfec3d 
Forward-merge branch-23.12 to branch-24.02
@GPUtester
Merge pull request #1299 from rapidsai/branch-23.12
4e37f64 
Forward-merge branch-23.12 to branch-24.02
@GPUtester
Merge pull request #1302 from rapidsai/branch-23.12
b7f0311 
Forward-merge branch-23.12 to branch-24.02
@GPUtester
Merge pull request #1306 from rapidsai/branch-23.12
13b297f 
Forward-merge branch-23.12 to branch-24.02
@vyasr
Remove CUSPATIAL_BUILD_WHEELS and standardize Python builds (#1304)
feaf99e 
Some minor simplification in advance of the scikit-build-core migration to better align wheel and non-wheel Python builds. Additionally, this PR simplifies handling of the nvcomp targets and their installation.

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

Approvers:
  - Mark Harris (https://github.com/harrism)
  - Ray Douglass (https://github.com/raydouglass)

URL: #1304
@vyasr
Update dependencies.yaml to new pip index (#1305)
289267f 
This PR changes all references to pypi.nvidia.com to pypi.anaconda.org/rapidsai-wheels-nightly.

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

Approvers:
  - Mark Harris (https://github.com/harrism)
  - Ray Douglass (https://github.com/raydouglass)

URL: #1305
@GPUtester
Merge pull request #1311 from rapidsai/branch-23.12
1602638 
Forward-merge branch-23.12 to branch-24.02
@bdice
Use Cython 3 for cuproj. (#1309)
cdccd49 
This updates cuproj's conda recipe to use `cython >=3.0.0` to align with other RAPIDS packages.

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

Approvers:
  - Mark Harris (https://github.com/harrism)
  - https://github.com/jakirkham
  - Ray Douglass (https://github.com/raydouglass)

URL: #1309
@vyasr
Switch to scikit-build-core (#1313)
60de429 
Contributes to rapidsai/build-planning#2

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

Approvers:
  - Bradley Dice (https://github.com/bdice)
  - AJ Schmidt (https://github.com/ajschmidt8)
  - Mark Harris (https://github.com/harrism)

URL: #1313
@bdice
Use cuda::proclaim_return_type on device lambdas. (#1314)
8e8fa06 
This PR makes cuspatial compatible with Thrust 2 by adding `cuda::proclaim_return_type`, but does not upgrade the version of Thrust/CCCL just yet. Currently we use libcudacxx 2.1.0, which makes `cuda::proclaim_return_type` available, but we still use Thrust 1.17.2 which doesn't require device lambdas to have proclaimed return types. This diff is separated out from #1096. I will create a separate PR which contains only packaging changes / version updates for the CCCL 2 migration. This PR is **nonbreaking**, while the second PR will be **breaking**.

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

Approvers:
  - Mark Harris (https://github.com/harrism)
  - Michael Wang (https://github.com/isVoid)

URL: #1314
@bdice
Update to CCCL 2.2.0. (#1315)
9501982 
This PR updates cuspatial to CCCL 2.2.0. Do not merge until all of RAPIDS is ready to update.

Follow-up to #1314.

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

Approvers:
  - Vyas Ramasubramani (https://github.com/vyasr)
  - Ray Douglass (https://github.com/raydouglass)
  - Robert Maynard (https://github.com/robertmaynard)
  - Mark Harris (https://github.com/harrism)

URL: #1315
@jameslamb
refactor CUDA versions in dependencies.yaml (#1317)
48f601b 
Contributes to rapidsai/build-planning#7.

Proposes splitting the `cuda-version` dependency in `dependencies.yaml` out to its own thing, separate from the bits of the CUDA Toolkit this project needs.

### Benefits of this change

* prevents accidental inclusion of multiple `cuda-version` version in environments
* reduces update effort (via enabling more use of globs like `"12.*"`)
* improves the chance that errors like "`conda` recipe is missing a dependency" are caught in CI

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

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

URL: #1317
@KyleFromNVIDIA
Remove usages of rapids-env-update (#1321)
36549cd 
Reference: rapidsai/ops#2766

Replace rapids-env-update with rapids-configure-conda-channels,
rapids-configure-sccache, and rapids-date-string.

Authors:
   - Kyle Edwards (https://github.com/KyleFromNVIDIA)

Approvers:
   - AJ Schmidt (https://github.com/ajschmidt8)
@AjayThorve
Update usage of cudf.core.column.arange to cudf.core.column.as_column (…
9b2b723 
…#1323)

This PR fixes failures which were caused becuase cuspatial uses an internal method of cudf that was removed in rapidsai/cudf#14689. The errors can be fixed by replacing those usages in the same fashion as was done internally in that cudf PR.

cc @vyasr

Authors:
  - Ajay Thorve (https://github.com/AjayThorve)

Approvers:
  - Michael Wang (https://github.com/isVoid)

URL: #1323
@harrism
Add explicit initial size to all pools and avoid RMM detail APIs (#1319)
43bb8f3 
This PR fixes up cuSpatial to avoid usage that will soon be deprecated in RMM.

Depends on rapidsai/rmm#1417

Fixes #1318

Authors:
  - Mark Harris (https://github.com/harrism)

Approvers:
  - Michael Wang (https://github.com/isVoid)

URL: #1319
@mroeschke
Replace cudf _from_columns with a public API (#1326)
534b267 
This private API will be removed in 24.02

Authors:
  - Matthew Roeschke (https://github.com/mroeschke)

Approvers:
  - Michael Wang (https://github.com/isVoid)
  - Mark Harris (https://github.com/harrism)

URL: #1326
@mroeschke
Use ListColumn instead of build_list_column (#1327)
6113f5f 
This API is planned for removal in cudf: rapidsai/cudf#14786

Authors:
  - Matthew Roeschke (https://github.com/mroeschke)

Approvers:
  - Mark Harris (https://github.com/harrism)
  - Michael Wang (https://github.com/isVoid)

URL: #1327
@vyasr
Make sure cudf uses pyarrow's libarrow during wheel builds (#1334)
d185528 
This fixes an error introduced in #1304.

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

Approvers:
  - Mark Harris (https://github.com/harrism)
  - Paul Taylor (https://github.com/trxcllnt)
  - Ray Douglass (https://github.com/raydouglass)

URL: #1334
@raydouglass raydouglass requested review from a team as code owners February 2, 2024 21:40
@review-notebook-app Review Notebook App
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@github-actions github-actions bot added conda Related to conda and conda configuration cmake Related to CMake code or build configuration Python Related to Python code libcuspatial Relates to the cuSpatial C++ library labels Feb 2, 2024
@github-actions github-actions bot added the ci label Feb 2, 2024
@raydouglass raydouglass merged commit 75adb4c into main Feb 13, 2024
66 of 79 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@harrism harrism Awaiting requested review from harrism harrism is a code owner automatically assigned from rapidsai/cuspatial-cmake-codeowners

@trxcllnt trxcllnt Awaiting requested review from trxcllnt trxcllnt is a code owner automatically assigned from rapidsai/cuspatial-python-codeowners

Assignees
No one assigned
Labels
ci cmake Related to CMake code or build configuration conda Related to conda and conda configuration libcuspatial Relates to the cuSpatial C++ library Python Related to Python code
Projects
cuSpatial
Status: Todo
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@raydouglass @GPUtester @vyasr @bdice @jameslamb @KyleFromNVIDIA @AjayThorve @harrism @mroeschke