Skip to content

Fix Leiden refinement phase#3990

Merged
rapids-bot[bot] merged 21 commits intorapidsai:branch-23.12from
naimnv:fix_leiden
Nov 20, 2023
Merged

Fix Leiden refinement phase#3990
rapids-bot[bot] merged 21 commits intorapidsai:branch-23.12from
naimnv:fix_leiden

Conversation

@naimnv
Copy link
Contributor

@naimnv naimnv commented Nov 8, 2023 •
edited
Loading

  • Normalization factor was missing in the equation to decide if a node and a refined community is strongly connected inside their Louvain community. This PR adds that factor.
  • Disable random moves in the refinement phase. We plan to expose a flag to enable/disable random moves in a future PR.
  • Adds new function to flatten Leiden dendrogram as dendrogram flattening process needs additional info to unroll hierarchical leiden clustering

Closes #3850
Closes #3749

@naimnv naimnv requested a review from a team as a code owner November 8, 2023 17:44
@copy-pr-bot
Copy link

copy-pr-bot bot commented Nov 8, 2023

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@naimnv naimnv added bug Something isn't working non-breaking Non-breaking change labels Nov 8, 2023
@naimnv naimnv self-assigned this Nov 8, 2023
ChuckHastings
ChuckHastings reviewed Nov 8, 2023
View reviewed changes
@naimnv
Copy link
Contributor Author

naimnv commented Nov 8, 2023

/ok to test

seunghwak
seunghwak approved these changes Nov 8, 2023
View reviewed changes
Copy link
Contributor

@seunghwak seunghwak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but please use FIXME to better track issues we need to revisit.

@naimnv
Copy link
Contributor Author

naimnv commented Nov 8, 2023

/ok to test

@alexbarghi-nv
Copy link
Member

alexbarghi-nv commented Nov 12, 2023

/ok to test

@BradReesWork BradReesWork added this to the 23.12 milestone Nov 14, 2023
@ChuckHastings
Copy link
Collaborator

ChuckHastings commented Nov 16, 2023

/ok to test

@naimnv
Copy link
Contributor Author

naimnv commented Nov 16, 2023

/ok to test

@naimnv naimnv requested a review from a team as a code owner November 16, 2023 15:28
@naimnv
Copy link
Contributor Author

naimnv commented Nov 16, 2023

/ok to test

@naimnv
Copy link
Contributor Author

naimnv commented Nov 16, 2023

/ok to test

@naimnv
Copy link
Contributor Author

naimnv commented Nov 16, 2023

/ok to test

@naimnv
Copy link
Contributor Author

naimnv commented Nov 17, 2023

/ok to test

@naimnv
Copy link
Contributor Author

naimnv commented Nov 17, 2023

/ok to test

@naimnv
Copy link
Contributor Author

naimnv commented Nov 17, 2023

/ok to test

@naimnv
Copy link
Contributor Author

naimnv commented Nov 17, 2023

/ok to test

@naimnv
Copy link
Contributor Author

naimnv commented Nov 17, 2023

/ok to test

@naimnv
Copy link
Contributor Author

naimnv commented Nov 18, 2023

/ok to test

@ChuckHastings
Copy link
Collaborator

ChuckHastings commented Nov 20, 2023

/merge

@rapids-bot rapids-bot bot merged commit 8549b54 into rapidsai:branch-23.12 Nov 20, 2023
@eriknw eriknw mentioned this pull request Nov 27, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@BradReesWork BradReesWork BradReesWork approved these changes

@ChuckHastings ChuckHastings ChuckHastings approved these changes

@seunghwak seunghwak seunghwak approved these changes

Assignees

@naimnv naimnv

Labels

bug Something isn't working non-breaking Non-breaking change

Projects

None yet

Milestone

23.12

Development

Successfully merging this pull request may close these issues.

Several Possible Errors in Leiden Implementation Leiden Clustering produces too many clusters

5 participants

@naimnv @alexbarghi-nv @ChuckHastings @BradReesWork @seunghwak