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.

Sign up for GitHub

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

Jump to bottom

MNMG Louvain - Python #2493

Closed
Tracked by #3273
ChuckHastings opened this issue Aug 3, 2022 · 1 comment · Fixed by #2705
Closed
Tracked by #3273

MNMG Louvain - Python #2493

ChuckHastings opened this issue Aug 3, 2022 · 1 comment · Fixed by #2705
Assignees
@jnke2016
Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Milestone
22.10

Comments

@ChuckHastings
Copy link
Collaborator

The python implementation of Louvain still uses the legacy implementation. There is already an SG and MG Louvain available in PLC (pylibcugraph).

Existing tests should be sufficient, it should just be a matter of refactoring the python code.
This was referenced Aug 3, 2022
Closed
Closed
@rlratzel rlratzel added this to the 22.10 milestone Aug 10, 2022
@github-actions
Copy link

github-actions bot commented Sep 9, 2022

This issue has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed. This issue will be labeled inactive-90d if there is no activity in the next 60 days.

@jnke2016 jnke2016 mentioned this issue Sep 19, 2022
Merged
rapids-bot bot pushed a commit that referenced this issue Sep 30, 2022
@jnke2016
Refactor louvain (#2705)
854e51a 
This PR refactors Louvain by leveraging the C API and does minor updates to the python tests
closes #2493

Authors:
  - Joseph Nke (https://github.com/jnke2016)
  - Chuck Hastings (https://github.com/ChuckHastings)

Approvers:
  - Rick Ratzel (https://github.com/rlratzel)
  - Chuck Hastings (https://github.com/ChuckHastings)

URL: #2705
@kingmesal kingmesal mentioned this issue Feb 14, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jnke2016 jnke2016

Labels
improvement Improvement / enhancement to an existing function non-breaking Non-breaking change
Projects
None yet
Milestone
22.10
Development

Successfully merging a pull request may close this issue.

Refactor louvain jnke2016/cugraph
4 participants
@rlratzel @BradReesWork @ChuckHastings @jnke2016