Skip to content

Addressing Issue 5397 5384#5490

Merged
rapids-bot[bot] merged 14 commits intorapidsai:mainfrom
BradReesWork:issue_5397_5384
May 1, 2026
Merged

Addressing Issue 5397 5384#5490
rapids-bot[bot] merged 14 commits intorapidsai:mainfrom
BradReesWork:issue_5397_5384

Conversation

@BradReesWork
Copy link
Copy Markdown
Member

@BradReesWork BradReesWork commented Apr 13, 2026

  1. Addressed issues 5397 that batch ego graph was still being called after being drooped

  2. Dropped batch betweenness centrality

  3. Dropped use of python-louvain

  4. added some benchamrk tweaks to help ensure memory is collected

Close #5397
Close #5384

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Apr 13, 2026

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.

@BradReesWork BradReesWork added non-breaking Non-breaking change improvement Improvement / enhancement to an existing function labels Apr 14, 2026
@BradReesWork BradReesWork self-assigned this Apr 14, 2026
@BradReesWork BradReesWork marked this pull request as ready for review April 14, 2026 13:08
@BradReesWork BradReesWork requested a review from a team as a code owner April 14, 2026 13:09
@BradReesWork BradReesWork requested a review from acostadon April 14, 2026 14:38
Copy link
Copy Markdown
Contributor

@acostadon acostadon left a comment

Choose a reason for hiding this comment

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

Looks good

Comment thread benchmarks/cugraph/pytest-based/bench_algos.py Outdated
Comment thread python/cugraph/cugraph/tests/sampling/test_egonet_mg.py
Comment thread python/cugraph/cugraph/tests/community/test_louvain.py
@rlratzel
Copy link
Copy Markdown
Contributor

Sorry, I submitted my PR review without a summary:

Thanks, @BradReesWork for taking care of these items. I have a few requests which I already posted, LMK if I can help.

@BradReesWork BradReesWork requested a review from a team as a code owner April 21, 2026 14:39
@BradReesWork BradReesWork requested a review from gforsyth April 21, 2026 14:39
@rlratzel
Copy link
Copy Markdown
Contributor

rlratzel commented May 1, 2026

/merge

@rapids-bot rapids-bot Bot merged commit 946a56b into rapidsai:main May 1, 2026
197 of 212 checks passed
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 this pull request may close these issues.

[BUG]: batched_ego_graphs (deprecated and removed) still used in an MG test Drop dependence on python-louvain in testing

5 participants