Skip to content

Upgrade to nvcomp 5.1.0.21#20770

Merged
gforsyth merged 9 commits intorapidsai:mainfrom
bdice:nvcomp-5.1
Dec 3, 2025
Merged

Upgrade to nvcomp 5.1.0.21#20770
gforsyth merged 9 commits intorapidsai:mainfrom
bdice:nvcomp-5.1

Conversation

@bdice
Copy link
Copy Markdown
Contributor

@bdice bdice commented Dec 2, 2025

Description

Upgrades nvcomp to version 5.1.0.21.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Dec 2, 2025

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actions github-actions bot added the CMake CMake build issue label Dec 2, 2025
@bdice
Copy link
Copy Markdown
Contributor Author

bdice commented Dec 2, 2025

This depends on conda packages, which are not yet published.

@bdice bdice added DO NOT MERGE Hold off on merging; see PR for details improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Dec 2, 2025
@github-actions github-actions bot added the Python Affects Python cuDF API. label Dec 2, 2025
@GPUtester GPUtester moved this to In Progress in cuDF Python Dec 2, 2025
@KyleFromNVIDIA
Copy link
Copy Markdown
Member

/ok to test 52198c7

@KyleFromNVIDIA
Copy link
Copy Markdown
Member

/ok to test f682b07

@KyleFromNVIDIA
Copy link
Copy Markdown
Member

/ok to test 6b816e2

@KyleFromNVIDIA
Copy link
Copy Markdown
Member

/ok to test 73213c7

@KyleFromNVIDIA
Copy link
Copy Markdown
Member

/ok to test e52b7d3

@KyleFromNVIDIA KyleFromNVIDIA marked this pull request as ready for review December 3, 2025 14:45
@KyleFromNVIDIA KyleFromNVIDIA requested review from a team as code owners December 3, 2025 14:45
Copy link
Copy Markdown
Contributor Author

@bdice bdice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-approving. Thanks @KyleFromNVIDIA for picking this up!

@bdice bdice removed the DO NOT MERGE Hold off on merging; see PR for details label Dec 3, 2025
@bdice
Copy link
Copy Markdown
Contributor Author

bdice commented Dec 3, 2025

/merge

@gforsyth gforsyth merged commit 8605e83 into rapidsai:main Dec 3, 2025
47 of 49 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in cuDF Python Dec 3, 2025
@vyasr vyasr mentioned this pull request Dec 10, 2025
3 tasks
rapids-bot bot pushed a commit that referenced this pull request Dec 18, 2025
nvcomp exports need to be unconditional since #20770 switched us to using the nvcomp wheels.

I also removed the `rapids_export_find_package_root` call, which I believe is an artifact from the days of kvikio being header-only and being cloned into cudf. Now that we are usually finding a preexisting kvikio package we don't need this.

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

Approvers:
  - Nghia Truong (https://github.com/ttnghia)

URL: #20828
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CMake CMake build issue improvement Improvement / enhancement to an existing function non-breaking Non-breaking change Python Affects Python cuDF API.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants