Skip to content

Add vertex-based adjsets to to_polygonal#1545

Open
nselliott wants to merge 21 commits into
developfrom
feature/nselliott/polygonal-adjsets
Open

Add vertex-based adjsets to to_polygonal#1545
nselliott wants to merge 21 commits into
developfrom
feature/nselliott/polygonal-adjsets

Conversation

@nselliott

Copy link
Copy Markdown
Contributor

The implementation was missing the needed adjsets on the output of the transformation from structured AMR domains to the polygonal mesh.

@nselliott

Copy link
Copy Markdown
Contributor Author

I have put additional changes and features into this PR. These include:

  • Documentation and verification of the structured window adjset variant that I developed to enable the entire to_polytopal feature for AMR meshes. These adjsets describe adjacency in terms of the structured index space, rather than storing shared vertices.
  • Full generation of unstructured shared-index adjsets on the output mesh of to_polytopal.
  • A new generate_boundary function that adds an edge or face-based boundary topology to the unstructured mesh. This is not part of to_polytopal, but can be called afterward to add this topology when needed.
  • Handling of ranks that have no domains to provide to Partitioner::combine(), which can happen in AMR mesh flattening situations.

@nselliott nselliott marked this pull request as draft June 4, 2026 17:20
@nselliott

Copy link
Copy Markdown
Contributor Author

I'm not sure it's time to remove the Draft label, but it may be close, and folks can take a look at it.

@nselliott nselliott marked this pull request as ready for review June 10, 2026 20:36
@nselliott nselliott changed the title Draft: Add vertex-based adjsets to to_polygonal Add vertex-based adjsets to to_polygonal Jun 10, 2026
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.

1 participant