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

Add pylibcugraph dependency on pylibraft. #4570

Merged

Conversation

bdice
Copy link
Contributor

@bdice bdice commented Jul 30, 2024

@quasiben noticed that pylibcugraph was missing a pylibraft dependency. See #4568.

This PR adds the missing dependency on pylibraft, and also adds a dependency on rmm. This aligns with the wheel packaging:

cugraph/dependencies.yaml

Lines 132 to 139 in aa0347c

py_run_pylibcugraph:
output: pyproject
pyproject_dir: python/pylibcugraph
extras:
table: project
includes:
- depends_on_rmm
- depends_on_pylibraft

@bdice bdice requested a review from a team as a code owner July 30, 2024 20:30
@bdice bdice requested a review from AyodeAwe July 30, 2024 20:30
@github-actions github-actions bot added the conda label Jul 30, 2024
@rlratzel rlratzel added bug Something isn't working non-breaking Non-breaking change labels Jul 31, 2024
@rlratzel
Copy link
Contributor

/merge

@rapids-bot rapids-bot bot merged commit a0b112b into rapidsai:branch-24.08 Jul 31, 2024
131 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working conda non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants