Skip to content

Add VTU importer with surface + volume cell partitioning#234

Merged
jeffamstutz merged 1 commit intoNVIDIA:next_releasefrom
tarcila:vtu-importer
Feb 23, 2026
Merged

Add VTU importer with surface + volume cell partitioning#234
jeffamstutz merged 1 commit intoNVIDIA:next_releasefrom
tarcila:vtu-importer

Conversation

@tarcila
Copy link
Collaborator

@tarcila tarcila commented Feb 23, 2026

This improves on the VTU importer so:

  • it is providing SpatialField for the -volume command line importer
  • it is providing both SpatialField and surfaces when using the new -vtu

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request enhances the VTU importer to support both surface and volume cell partitioning, providing a more comprehensive import capability for VTK Unstructured Grid files. The changes enable the -vtu command-line option to import both surfaces (triangles, quads, polygons) and volumes (tetrahedra, voxels, hexahedra, wedges, pyramids) from a single VTU file, while maintaining backward compatibility with the -volume option that only imports volume cells as a SpatialField.

Changes:

  • Added new VTU importer with dual functionality: full scene import with surfaces + volumes, and spatial field-only import for backward compatibility
  • Implemented cell type partitioning to separate surface cells (for mesh rendering) from volume cells (for spatial field rendering)
  • Integrated the new importer into the command-line interface and file import dispatcher

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
tsd/src/tsd/io/importers/import_VTU.cpp Complete rewrite with surface extraction/triangulation and volume cell filtering logic
tsd/src/tsd/io/importers.hpp Added declaration for new void import_VTU(Scene&, const char*, LayerNodeRef) overload
tsd/src/tsd/io/importers/import_file.cpp Integrated VTU importer into the file dispatch logic
tsd/src/tsd/app/Core.cpp Added -vtu command-line flag handling

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jeffamstutz
Copy link
Collaborator

LGTM, thanks!

@jeffamstutz jeffamstutz merged commit 5dfd172 into NVIDIA:next_release Feb 23, 2026
13 checks passed
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.

3 participants