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 python Betweenness Centrailty to use C API #3145

Closed
4 tasks
Tracked by #3272
rlratzel opened this issue Jan 17, 2023 · 0 comments · Fixed by #2971
Closed
4 tasks
Tracked by #3272

Update python Betweenness Centrailty to use C API #3145

rlratzel opened this issue Jan 17, 2023 · 0 comments · Fixed by #2971
Assignees
Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Milestone

Comments

@rlratzel
Copy link
Contributor

BC is available in the C API, and pylibcugraph and cugraph should be using that instead of the legacy implementation in the python API.

  • Add BC to PLC
  • Update PLC tests, docs, etc.
  • Update cugraph to call BC in PLC
  • Remove legacy BC in cugraph
@rlratzel rlratzel added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jan 17, 2023
@rlratzel rlratzel added this to the 23.02 milestone Jan 17, 2023
rapids-bot bot pushed a commit that referenced this issue Mar 31, 2023
A CAPI implementation of betweenness centrality is available and this PR:
1. Implement PLC betweenness centrality by leveraging the CAPI
2. Refactor the python SG implementation of betweenness centrality by leveraging the PLC implementation
3. Add a python MG implementation of betweenness centrality with tests

closes #3145 
closes #2605
closes #2648 
closes #2649
closes #2650

Authors:
  - Joseph Nke (https://github.com/jnke2016)
  - Chuck Hastings (https://github.com/ChuckHastings)
  - Rick Ratzel (https://github.com/rlratzel)
  - Alex Barghi (https://github.com/alexbarghi-nv)
  - Vyas Ramasubramani (https://github.com/vyasr)
  - Vibhu Jawa (https://github.com/VibhuJawa)

Approvers:
  - Rick Ratzel (https://github.com/rlratzel)
  - Chuck Hastings (https://github.com/ChuckHastings)
  - Alex Barghi (https://github.com/alexbarghi-nv)
  - Brad Rees (https://github.com/BradReesWork)

URL: #2971
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants