Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/tbrom pyvista 24r2 part2 (stacked on feat/tbrom_pyvista_24r2) #143

Merged
merged 40 commits into from
Jan 22, 2024

Conversation

chrpetre
Copy link
Collaborator

@chrpetre chrpetre commented Jan 3, 2024

follow up of abandoned PR #126 (part 2) :

This PR introduces new capabilities regarding 3D field visualization

  • it adds new APIs (TBROM, TwinModel) to project field data onto given mesh, and update the resulting field with respect to current Twin states
  • associated unit testing
  • new examples illustrating the mesh projection workflow for CFD and FEA models

It adds the necessary dependencies and updates the CI workflow to install a DPF server when building the documentation and running the examples

@github-actions github-actions bot added documentation Improvements or additions to documentation maintenance Package and maintenance related dependencies Related with project dependencies enhancement New features or code improvements labels Jan 3, 2024
@codecov-commenter
Copy link

codecov-commenter commented Jan 3, 2024

Codecov Report

Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (0633050) 87.56% compared to head (f241991) 87.69%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #143      +/-   ##
==========================================
+ Coverage   87.56%   87.69%   +0.12%     
==========================================
  Files          11       11              
  Lines        2461     2543      +82     
==========================================
+ Hits         2155     2230      +75     
- Misses        306      313       +7     

skip the 'F401 'tqdm.tqdm' imported but unused' error from flake8
@chrpetre chrpetre mentioned this pull request Jan 9, 2024
Base automatically changed from feat/tbrom_pyvista_24r2 to main January 11, 2024 13:46
Copy link
Collaborator

@lboucin lboucin left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks. Added important comments to be discussed.

src/ansys/pytwin/evaluate/twin_model.py Outdated Show resolved Hide resolved
src/ansys/pytwin/evaluate/twin_model.py Outdated Show resolved Hide resolved
src/ansys/pytwin/evaluate/twin_model.py Outdated Show resolved Hide resolved
src/ansys/pytwin/evaluate/twin_model.py Outdated Show resolved Hide resolved
src/ansys/pytwin/evaluate/twin_model.py Show resolved Hide resolved
@chrpetre chrpetre merged commit ae3288d into main Jan 22, 2024
23 checks passed
@chrpetre chrpetre deleted the feat/tbrom_pyvista_24r2_part2 branch January 22, 2024 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Related with project dependencies documentation Improvements or additions to documentation enhancement New features or code improvements maintenance Package and maintenance related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants