Skip to content

ENH: Consistency in wrapped mesh types#403

Merged
thewtex merged 1 commit into
InsightSoftwareConsortium:masterfrom
thewtex:wrapped-mesh-pixel-types
Jan 13, 2019
Merged

ENH: Consistency in wrapped mesh types#403
thewtex merged 1 commit into
InsightSoftwareConsortium:masterfrom
thewtex:wrapped-mesh-pixel-types

Conversation

@thewtex
Copy link
Copy Markdown
Member

@thewtex thewtex commented Jan 12, 2019

This patch simplifies, uses consistently, and expand pixel types
supported for mesh processing with Python wrapping.

We now do not explicitly specify the mesh traits used. The
itk::DefaultStaticMeshTraits with default template parameters are used.
This serves all current use cases. The use of the default traits greatly
improves the Python user experience and makes it easier to wrap
additional classes.

More pixel types for the PointSet point and Mesh cells are wrapped --
all enabled wrapping scalar pixel types verses just the real types.
These types are used consistently across the wrapped classes.

This patch simplifies, uses consistently, and expand pixel types
supported for mesh processing with Python wrapping.

We now do not explicitly specify the mesh traits used. The
itk::DefaultStaticMeshTraits with default template parameters are used.
This serves all current use cases. The use of the default traits greatly
improves the Python user experience and makes it easier to wrap
additional classes.

More pixel types for the PointSet point and Mesh cells are wrapped --
all enabled wrapping scalar pixel types verses just the real types.
These types are used consistently across the wrapped classes.
Copy link
Copy Markdown
Contributor

@phcerdan phcerdan left a comment

Choose a reason for hiding this comment

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

Looks great!

Comment thread Modules/Core/Mesh/wrapping/itkMeshBase.wrap
Copy link
Copy Markdown
Member

@jhlegarreta jhlegarreta left a comment

Choose a reason for hiding this comment

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

I'm speechless. Amazing.

Copy link
Copy Markdown
Member

@dzenanz dzenanz left a comment

Choose a reason for hiding this comment

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

Will this help InsightSoftwareConsortium/ITKSphinxExamples#66 roll forward?

@thewtex
Copy link
Copy Markdown
Member Author

thewtex commented Jan 13, 2019

Will this help InsightSoftwareConsortium/ITKExamples#66 roll forward?

Yes :-D

@thewtex thewtex merged commit 4f6b3d8 into InsightSoftwareConsortium:master Jan 13, 2019
@thewtex thewtex deleted the wrapped-mesh-pixel-types branch January 13, 2019 13:22
hjmjohnson pushed a commit to hjmjohnson/ITK that referenced this pull request May 6, 2026
…-mesh-pixel-types

ENH: Consistency in wrapped mesh types
hjmjohnson pushed a commit to hjmjohnson/ITK that referenced this pull request May 12, 2026
…-mesh-pixel-types

ENH: Consistency in wrapped mesh types
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.

4 participants