Skip to content

Adding distributed orthogonal mesh creation.#1070

Open
wdhawkins wants to merge 1 commit into
Open-Sn:mainfrom
wdhawkins:distributed_ortho_mesh
Open

Adding distributed orthogonal mesh creation.#1070
wdhawkins wants to merge 1 commit into
Open-Sn:mainfrom
wdhawkins:distributed_ortho_mesh

Conversation

@wdhawkins

@wdhawkins wdhawkins commented May 14, 2026

Copy link
Copy Markdown
Collaborator

This PR adds distributed orthogonal mesh creation. This implementation differs from true distributed mesh creation in that rank 0 creates the entire global cell graph. This was intentional, as it allows the existing partitioning algorithms to work as is. After partitioning, rank 0 distributes partition and cell data to each rank, and the individual ranks build their own cells and populate the mesh.

Once this is tested at scale, I'll remove the "Draft" designation.

Closes #19

PR Checklist

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

@wdhawkins wdhawkins self-assigned this May 14, 2026
@wdhawkins wdhawkins added the enhancement New feature or request label May 14, 2026
@wdhawkins wdhawkins force-pushed the distributed_ortho_mesh branch from 51799cd to 74fceae Compare May 14, 2026 15:56
@ragusa

ragusa commented May 14, 2026

Copy link
Copy Markdown
Contributor

should there be an example/tutorial and/or a regression test added?

@wdhawkins

Copy link
Copy Markdown
Collaborator Author

should there be an example/tutorial and/or a regression test added?

Yes. I'll add regression tests and a tutorial.

@ragusa ragusa left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

read the comments and decide. you can just resolve the conversation if appropriate.

Comment thread python/lib/mesh.cc

@andrsd andrsd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This seems ok.
It is marked as draft, so I can check again when ready for review.

@wdhawkins wdhawkins force-pushed the distributed_ortho_mesh branch 3 times, most recently from cecfd7c to 2d76d59 Compare May 30, 2026 04:04
@wdhawkins wdhawkins force-pushed the distributed_ortho_mesh branch from 2d76d59 to c9da699 Compare June 4, 2026 04:36
@wdhawkins wdhawkins marked this pull request as ready for review June 8, 2026 17:18
@wdhawkins wdhawkins requested a review from andrsd June 13, 2026 23:30
const int cell_pid = cell_pids[cell_gid];
++partI_num_cells[cell_pid];

std::array<int, 27> recipient_pids = {};

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Maybe a note on the 27?

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SplitMesh

3 participants