Skip to content

Comments

feat: process and plot contour over a mesh during pv to mesh3d conversion#450

Merged
AlejandroFernandezLuces merged 3 commits intomainfrom
feat/process-and-plot-viz-data-during-pv-to-mesh3d-conversion
Jan 12, 2026
Merged

feat: process and plot contour over a mesh during pv to mesh3d conversion#450
AlejandroFernandezLuces merged 3 commits intomainfrom
feat/process-and-plot-viz-data-during-pv-to-mesh3d-conversion

Conversation

@moe-ad
Copy link
Contributor

@moe-ad moe-ad commented Jan 12, 2026

For a pyvista mesh with some active dataset (expected to be used for the contour plot over the mesh), the pvmesh-mesh3d conversion taking place behind the scenes discards the data and simply plots the mesh.

Point data visualization is supported by plotly, so we can at least try to detect if the mesh being converted has an active point dataset, do some processing, and plot a contour over the mesh using it.

Before:

Image

After

Image

PS: I am not sure if this is the most robust way to handle this though. Maybe it requires more thought @AlejandroFernandezLuces ?

@github-actions github-actions bot added the enhancement New features or code improvements label Jan 12, 2026
Copy link
Collaborator

@AlejandroFernandezLuces AlejandroFernandezLuces left a comment

Choose a reason for hiding this comment

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

From my point of view, there is probably more missing information that can be contained in the VTK object. Let's merge and keep adding features as they are needed.

@AlejandroFernandezLuces AlejandroFernandezLuces merged commit 146f858 into main Jan 12, 2026
23 checks passed
@AlejandroFernandezLuces AlejandroFernandezLuces deleted the feat/process-and-plot-viz-data-during-pv-to-mesh3d-conversion branch January 12, 2026 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New features or code improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants