Skip to content

FEAT: Add plot option for mesh - #1268

Merged
koubaa merged 18 commits into
mainfrom
plot-mesh
Aug 7, 2025
Merged

FEAT: Add plot option for mesh#1268
koubaa merged 18 commits into
mainfrom
plot-mesh

Conversation

@koubaa

@koubaa koubaa commented Aug 6, 2025

Copy link
Copy Markdown
Contributor

Use new mechanical API to get the scenegraph for meshes and results

Add handling for line bodies, point clouds, arbitrary scenegraph structure, and MeshOrientedTransformNode (under an env var, it isn't correct)

To render a mesh:
app.plot(app.Model.Mesh)

To render a result:

app.plot(app.Model.GetChildren(DataModelObjectCategory.Result, True)[0])

@koubaa
koubaa requested a review from dipinknair August 6, 2025 19:51
@koubaa
koubaa requested a review from a team as a code owner August 6, 2025 19:51
@codacy-production

codacy-production Bot commented Aug 6, 2025

Copy link
Copy Markdown

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
-0.03% (target: -1.00%) 40.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (c152a6c) 2544 2187 85.97%
Head commit (ebf3d10) 2545 (+1) 2187 (+0) 85.93% (-0.03%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#1268) 5 2 40.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

@codecov

codecov Bot commented Aug 6, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 40.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.93%. Comparing base (c152a6c) to head (ebf3d10).
⚠️ Report is 1 commits behind head on main.

❌ Your patch status has failed because the patch coverage (40.00%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1268      +/-   ##
==========================================
- Coverage   85.96%   85.93%   -0.04%     
==========================================
  Files          39       39              
  Lines        2544     2545       +1     
==========================================
  Hits         2187     2187              
- Misses        357      358       +1     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dipinknair dipinknair changed the title Plot mesh FEAT: Add plot option for mesh Aug 6, 2025
@dipinknair

Copy link
Copy Markdown
Contributor

@koubaa let me test this on examples.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Aug 6, 2025
@koubaa
koubaa merged commit 1e68b8b into main Aug 7, 2025
59 of 61 checks passed
@koubaa
koubaa deleted the plot-mesh branch August 7, 2025 13:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants