Skip to content

Host CBC handles cycles#1060

Open
eappen-nelluvelil wants to merge 4 commits into
Open-Sn:mainfrom
eappen-nelluvelil:cbc-cycles-4
Open

Host CBC handles cycles#1060
eappen-nelluvelil wants to merge 4 commits into
Open-Sn:mainfrom
eappen-nelluvelil:cbc-cycles-4

Conversation

@eappen-nelluvelil

@eappen-nelluvelil eappen-nelluvelil commented May 9, 2026

Copy link
Copy Markdown
Contributor

PR Checklist

  • I have updated the user guide and/or Python API documentation if necessary.

Note: This PR builds on top of PR #1008 and PR #1050. Once 1008 and 1050 have been merged, only the latest 2 commits will be retained.

This PR updates host CBC's SPDS, FLUDS, and async communicator so FAS edges in local/inter-partition cycles are treated as lagged dependencies, and CBC_FLUDS manages additional delayed flux banks.

Device CBC support for handling cyclic dependencies was not added here. PR #1028 will be updated to allow CBCD V2 to handle cyclic dependencies.

strong_cbc_rz_vs_cbc_cycles_4 weak_cbc_rz_vs_cbc_cycles_4

Comment thread framework/mesh/cell/cell.h
@andrsd

andrsd commented May 13, 2026

Copy link
Copy Markdown
Collaborator

I looked over the code. It seems fairly good to me.

Also, it seems like you need some more wrappers in mpicpp-lite. I can add those and ping you here when done...

@eappen-nelluvelil eappen-nelluvelil force-pushed the cbc-cycles-4 branch 4 times, most recently from facaf75 to 642fede Compare May 21, 2026 20:26
@andrsd andrsd mentioned this pull request May 22, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants