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

Use nvcomp conda package. #245

Merged
merged 4 commits into from
Jun 26, 2023
Merged

Conversation

bdice
Copy link
Contributor

@bdice bdice commented Jun 14, 2023

Description

This PR uses conda-forge packages of nvcomp rather than fetching a tarball. This means that the nvcomp binary should not be shipped in the kvikio conda package, but is instead listed as a dependency. This will reduce kvikio's conda package size.

Resolves #242.

@bdice
Copy link
Contributor Author

bdice commented Jun 14, 2023

This is probably blocked on rapidsai/cudf#13566 (comment), I will revisit this PR later but wanted to get the draft open for now.

@bdice bdice added DO NOT MERGE improvement Improves an existing functionality conda non-breaking Introduces a non-breaking change labels Jun 14, 2023
@bdice bdice self-assigned this Jun 26, 2023
@bdice bdice marked this pull request as ready for review June 26, 2023 18:53
@bdice bdice requested a review from a team as a code owner June 26, 2023 18:53
@jakirkham
Copy link
Member

Thanks Bradley! 🙏

Do we need to do anything with get_nvcomp.cmake?

@bdice
Copy link
Contributor Author

bdice commented Jun 26, 2023

Thanks Bradley! 🙏

Do we need to do anything with get_nvcomp.cmake?

Nope. No changes should be needed there. rapids_cpm_nvcomp does exactly what we want.

@bdice
Copy link
Contributor Author

bdice commented Jun 26, 2023

I verified that the Python build logs look correct. The conda output shows lines like Needed DSO lib/libnvcomp.so found in conda-forge::nvcomp-2.6.1-h0800d71_2 which indicates that packaging works as intended.

@bdice
Copy link
Contributor Author

bdice commented Jun 26, 2023

/merge

@rapids-bot rapids-bot bot merged commit 1ffe8b1 into rapidsai:branch-23.08 Jun 26, 2023
@jakirkham
Copy link
Member

Thanks Bradley! 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conda improvement Improves an existing functionality non-breaking Introduces a non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use nvcomp package
3 participants