-
Notifications
You must be signed in to change notification settings - Fork 310
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 degrees to C API #4212
Add degrees to C API #4212
Conversation
This LGTM. I'm fine if we want to add the remaining updates needed to PLC, then the updates to the MG python code all to this PR if it's easier. |
Approved too soon - I'll re-review and approve when this is out of Draft and the remaining updates are added.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (C/C++ part)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! I only reviewed the python code and I have a few requests and questions.
python/cugraph/cugraph/structure/graph_implementation/simpleDistributedGraph.py
Outdated
Show resolved
Hide resolved
python/cugraph/cugraph/structure/graph_implementation/simpleDistributedGraph.py
Outdated
Show resolved
Hide resolved
python/cugraph/cugraph/structure/graph_implementation/simpleDistributedGraph.py
Outdated
Show resolved
Hide resolved
python/cugraph/cugraph/structure/graph_implementation/simpleDistributedGraph.py
Outdated
Show resolved
Hide resolved
/merge |
Add new method
cugraph_degrees
to the C API to compute and return the degrees of vertices.Closes #4171