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

Update for raft logger changes #4835

Open
wants to merge 7 commits into
base: branch-25.02
Choose a base branch
from

Conversation

vyasr
Copy link
Contributor

@vyasr vyasr commented Dec 17, 2024

This PR updates cugraph to use raft's updated logger implementation using rapids-logger.

This PR requires rapidsai/raft#2530.

Contributes to rapidsai/build-planning#104

@vyasr vyasr added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Dec 17, 2024
@vyasr vyasr self-assigned this Dec 17, 2024
Copy link

copy-pr-bot bot commented Dec 17, 2024

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.

@vyasr
Copy link
Contributor Author

vyasr commented Dec 17, 2024

/ok to test

@vyasr
Copy link
Contributor Author

vyasr commented Dec 18, 2024

/ok to test

@vyasr
Copy link
Contributor Author

vyasr commented Dec 18, 2024

/ok to test

@vyasr
Copy link
Contributor Author

vyasr commented Dec 18, 2024

/ok to test

@vyasr
Copy link
Contributor Author

vyasr commented Dec 19, 2024

/ok to test

@vyasr
Copy link
Contributor Author

vyasr commented Dec 19, 2024

/ok to test

@vyasr vyasr added the DO NOT MERGE Hold off on merging; see PR for details label Dec 19, 2024
@vyasr vyasr marked this pull request as ready for review December 19, 2024 22:04
@vyasr vyasr requested review from a team as code owners December 19, 2024 22:04
@vyasr vyasr requested a review from jameslamb December 19, 2024 22:04
@vyasr vyasr changed the title Update for raft-logger changes Update for raft logger changes Dec 19, 2024
@vyasr
Copy link
Contributor Author

vyasr commented Dec 19, 2024

The devcontainer failures are expected since I haven't set that case up to pull the conda packages from the raft PR. That won't be tested until we merge that one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci CMake cuGraph DO NOT MERGE Hold off on merging; see PR for details improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants