Skip to content

Replace legacy SCC with the new SCC implementation and SCC performance improvements#5469

Open
seunghwak wants to merge 20 commits intorapidsai:mainfrom
seunghwak:enh_scc_perf
Open

Replace legacy SCC with the new SCC implementation and SCC performance improvements#5469
seunghwak wants to merge 20 commits intorapidsai:mainfrom
seunghwak:enh_scc_perf

Conversation

@seunghwak
Copy link
Copy Markdown
Contributor

@seunghwak seunghwak commented Mar 18, 2026

Replace legacy SCC with the new SCC implementation.

In addition, this code adds chain removal logic (a chain of in-degree 1 out-degree 1 vertices) in peeling out trivial singleton SCC vertices. The chain removal is based on pointer jumping and significantly reduces the number of iterations in recursively removing 0 in-degree or 0 out-degree vertices.

Breaking as the legacy SCC is removed.

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Mar 18, 2026

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.

@seunghwak seunghwak marked this pull request as ready for review March 30, 2026 17:43
@seunghwak seunghwak requested review from a team as code owners March 30, 2026 17:43
@seunghwak seunghwak added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Mar 30, 2026
@seunghwak seunghwak changed the title [WIP] Replace legacy SCC with the new SCC implementation and SCC performance improvements Replace legacy SCC with the new SCC implementation and SCC performance improvements Mar 30, 2026
@seunghwak seunghwak added breaking Breaking change and removed non-breaking Non-breaking change labels Mar 30, 2026
@seunghwak seunghwak self-assigned this Mar 30, 2026
@ChuckHastings
Copy link
Copy Markdown
Collaborator

/ok to test 2bd8c9b

@seunghwak
Copy link
Copy Markdown
Contributor Author

/ok to test 5959301

@seunghwak
Copy link
Copy Markdown
Contributor Author

/ok to test ccc3fdc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Breaking change improvement Improvement / enhancement to an existing function

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants