Skip to content

Conversation

mauigna06
Copy link
Contributor

@mauigna06 mauigna06 commented Jul 21, 2025

Fixes #81

2025-08-12.15-41-09.mp4

@mauigna06 mauigna06 marked this pull request as ready for review July 21, 2025 21:07
@mauigna06
Copy link
Contributor Author

We just need an icon for it, other than that it works fine

@mauigna06 mauigna06 changed the title Subdivision dynamic modeler tool ENH: Subdivision dynamic modeler tool Jul 23, 2025
@mauigna06
Copy link
Contributor Author

One detail, none of the subdivision filters on this dynamic modeler tool support meshes with non-manifold edges.

For example, the python console will print the text below in case the input model is non-manifold and there will be no Slicer crash

[VTK] Warning: In vtkLoopSubdivisionFilter.cxx, line 150
[VTK] vtkLoopSubdivisionFilter (0x55f0a24a70b0): numCellsInLoop < 1: 0
[VTK] Subdivision failed.
[VTK] Algorithm vtkLoopSubdivisionFilter (0x55f0a24a70b0) returned failure for request: vtkInformation (0x55f0a317e700)
[VTK]   Debug: Off
[VTK]   Modified Time: 404687
[VTK]   Reference Count: 1
[VTK]   Registered Events: (none)
[VTK]   Request: REQUEST_DATA
[VTK]   FROM_OUTPUT_PORT: 0
[VTK]   ALGORITHM_AFTER_FORWARD: 1
[VTK]   FORWARD_DIRECTION: 0

@mauigna06
Copy link
Contributor Author

We just need an icon for it, other than that it works fine

Icon added, ready for review

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

Successfully merging this pull request may close these issues.

Subdivision feature
1 participant