Skip to content

Conversation

@PProfizi
Copy link
Contributor

@PProfizi PProfizi commented May 22, 2024

This PR aims at exposing helpers to convert DPF object instances to VTK object instances.

  • MeshedRegion
  • MeshesContainer
  • Field
  • FieldsContainer
  • PropertyField

@PProfizi PProfizi added the enhancement New feature or request label May 22, 2024
@PProfizi PProfizi added this to the v0.12.2 milestone May 22, 2024
@PProfizi PProfizi self-assigned this May 22, 2024
@codecov-commenter
Copy link

codecov-commenter commented May 22, 2024

Codecov Report

Attention: Patch coverage is 86.40777% with 14 lines in your changes are missing coverage. Please review.

Project coverage is 84.22%. Comparing base (7d62227) to head (5e5fc3b).
Report is 13 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1581      +/-   ##
==========================================
- Coverage   84.26%   84.22%   -0.04%     
==========================================
  Files          82       82              
  Lines        9671     9795     +124     
==========================================
+ Hits         8149     8250     +101     
- Misses       1522     1545      +23     

@PProfizi PProfizi changed the title Feat/dpf to vtk Add vtk_helpers May 28, 2024
@PProfizi PProfizi changed the title Add vtk_helpers Add vtk_helpers for DPF to VTK translation May 28, 2024
Signed-off-by: paul.profizi <[email protected]>
Copy link

@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.

PyVista/VTK wise, this PR looks good to me 🙂

This will be great for the integration of ansys-tool-visualization-interface at some point.

@PProfizi PProfizi merged commit 918c515 into master May 31, 2024
@PProfizi PProfizi deleted the feat/dpf_to_vtk branch May 31, 2024 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants