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

Fix kvikio casing and remove unnecessary rmm dependency #414

Merged
merged 3 commits into from
Nov 4, 2024

Conversation

vyasr
Copy link
Contributor

@vyasr vyasr commented Nov 4, 2024

The case-sensitive name KvikIO is throwing off find_package searches in various places.

@vyasr vyasr added bug Something isn't working non-breaking Introduces a non-breaking change labels Nov 4, 2024
@vyasr vyasr self-assigned this Nov 4, 2024
@vyasr vyasr requested a review from a team as a code owner November 4, 2024 22:03
@vyasr vyasr requested review from raydouglass and removed request for a team November 4, 2024 22:03
@vyasr
Copy link
Contributor Author

vyasr commented Nov 4, 2024

Merging without CI passing because there are interdependencies between this PR, rapidsai/cudf#17243, and rapidsai/kvikio#534.

@vyasr vyasr merged commit 519de02 into rapidsai:branch-24.12 Nov 4, 2024
22 of 26 checks passed
@vyasr vyasr deleted the fix/kvikio_casing branch November 4, 2024 22:32
rapids-bot bot pushed a commit to rapidsai/kvikio that referenced this pull request Nov 5, 2024
AyodeAwe pushed a commit to rapidsai/cudf that referenced this pull request Nov 6, 2024
## Description
<!-- Provide a standalone description of changes in this PR. -->
<!-- Reference any issues closed by this PR with "closes #1234". -->
<!-- Note: The pull request title will be included in the CHANGELOG. -->
The case-sensitive name KvikIO is will throw off `find_package`
searches, particularly after
rapidsai/devcontainers#414 make the usage
consistent in devcontainers.

## Checklist
- [x] I am familiar with the [Contributing
Guidelines](https://github.com/rapidsai/cudf/blob/HEAD/CONTRIBUTING.md).
- [x] New or existing tests cover these changes.
- [x] The documentation is up to date with these changes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working non-breaking Introduces a non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants