Skip to content

Read VTK_POLYHEDRON cells in the VTU reader - #4556

Open
GiudGiud wants to merge 1 commit into
libMesh:develfrom
GiudGiud:PR_vtu_reader
Open

GiudGiud wants to merge 1 commit into
libMesh:develfrom
GiudGiud:PR_vtu_reader

Conversation

@GiudGiud

@GiudGiud GiudGiud commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

refs #4555

did not do polygons

VTKIO::read() now handles VTK_POLYHEDRON cells, which VTK describes by a face stream rather than an ordered node list. Each such cell is built as a C0Polyhedron from C0Polygon faces (obtained via
vtkUnstructuredGrid::GetFaceStream), mirroring the Exodus NFACED reader; the interior mid-element node the C0Polyhedron construction may create is added to the mesh.

Adds a testVTKReadPolyhedra unit test that reads a hexagonal-prism polyhedron .vtu fixture and checks the element type, side and face connectivity.

refs libMesh#4555

VTKIO::read() now handles VTK_POLYHEDRON cells, which VTK describes by a
face stream rather than an ordered node list.  Each such cell is built as
a C0Polyhedron from C0Polygon faces (obtained via
vtkUnstructuredGrid::GetFaceStream), mirroring the Exodus NFACED reader;
the interior mid-element node the C0Polyhedron construction may create is
added to the mesh.

Adds a testVTKReadPolyhedra unit test that reads a hexagonal-prism
polyhedron .vtu fixture and checks the element type, side and face
connectivity.

Co-Authored-By: Claude <noreply@anthropic.com>
@GiudGiud
GiudGiud marked this pull request as ready for review September 17, 2026 13:29
@moosebuild

Copy link
Copy Markdown

Job Coverage, step Generate coverage on e46269f wanted to post the following:

Coverage

54e0d5 #4556 e46269
Total Total +/- New
Rate 65.99% 65.99% -0.00% -
Hits 79562 79559 -3 0
Misses 40996 40999 +3 0

Diff coverage report

Full coverage report

This comment will be updated on new commits.

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