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.

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

Upgrade nvcomp to 4.0.1 #633

Merged
merged 19 commits into from
Sep 5, 2024
Merged

Conversation

vuule
Copy link
Contributor

@vuule vuule commented Jun 24, 2024

Description

Upgrade to nvcomp 4.0. As part of that change we need to refactor the nvcomp.cmake to be aware of the new CMake files and targets that nvcomp generates.

Checklist

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

@vuule vuule added feature request New feature or request non-breaking Introduces a non-breaking change labels Jun 24, 2024
@robertmaynard
Copy link
Contributor

URL form for nvcomp 4.0 has changed we need nvcomp-linux-sbsa-4.0.0-cuda12.5.tar.gz instead of nvcomp_4.0.0_SBSA_12.x.tgz

Likewise for x86 which is now nvcomp-linux-x86_64-4.0.0-cuda12.5.tar.gz

@robertmaynard
Copy link
Contributor

Going to look at the changes required in rapids-cmake to support these new url strings later today

@robertmaynard robertmaynard self-assigned this Jul 19, 2024
@robertmaynard
Copy link
Contributor

More changes are required as the nvcomp 4.0 cmake package has a different set of files which needs to be patched for a proper install layout

@robertmaynard robertmaynard changed the title [test] upgrade nvcomp to 4.0 Upgrade nvcomp to 4.0 Jul 22, 2024
@robertmaynard robertmaynard marked this pull request as ready for review July 22, 2024 13:50
@robertmaynard robertmaynard requested a review from a team as a code owner July 22, 2024 13:50
@robertmaynard robertmaynard added the 5 - DO NOT MERGE Hold off on merging; see PR for details label Jul 22, 2024
@robertmaynard
Copy link
Contributor

Waiting on https://github.com/rapidsai/cudf/pull/16076/files being ready to merge

@robertmaynard
Copy link
Contributor

Blocked due to nvcomp now having a hard libcuda.so dependency.

@robertmaynard robertmaynard changed the base branch from branch-24.08 to branch-24.10 August 7, 2024 13:52
@vuule vuule changed the title Upgrade nvcomp to 4.0 Upgrade nvcomp to 4.0.1 Aug 27, 2024
Copy link
Contributor

@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.

One question, otherwise this looks good to me.

rapids-cmake/cpm/nvcomp.cmake Outdated Show resolved Hide resolved
@bdice bdice removed the 5 - DO NOT MERGE Hold off on merging; see PR for details label Sep 5, 2024
@bdice
Copy link
Contributor

bdice commented Sep 5, 2024

/merge

@rapids-bot rapids-bot bot merged commit 3c9bed0 into rapidsai:branch-24.10 Sep 5, 2024
15 checks passed
rapids-bot bot pushed a commit to rapidsai/kvikio that referenced this pull request Sep 5, 2024
Update the nvCOMP version used for compression/decompression to 4.0.1.

See also:

rapidsai/rapids-cmake#633
rapidsai/cudf#16076

Depends on #450.

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

Approvers:
  - Mads R. B. Kristensen (https://github.com/madsbk)
  - Bradley Dice (https://github.com/bdice)

URL: #449
rapids-bot bot pushed a commit to rapidsai/cudf that referenced this pull request Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request non-breaking Introduces a non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants