Skip to content

chore: switch default plotting backend to viz interface#3023

Merged
moe-ad merged 33 commits into
mainfrom
test/switch-tests-to-viz-interface
Mar 9, 2026
Merged

chore: switch default plotting backend to viz interface#3023
moe-ad merged 33 commits into
mainfrom
test/switch-tests-to-viz-interface

Conversation

@moe-ad
Copy link
Copy Markdown
Collaborator

@moe-ad moe-ad commented Mar 2, 2026

Follow up to #2881.

Summary of tasks done:

  • ansys-tools-visualization-interface is now added to plotting/graphics targets and doc/test dependency groups. This effectively switches the default backend to the viz interface backend.
  • Tests have been rearranged in a way that makes them pass, outside this new order, tests seg fault a ton.
  • Docstring tests have been updated.
  • A fix was applied to _VisualizationInterfacePlotter.add_point_labels.
  • Documentation updates made in doc/conf.py(as a workaround for labels not being captured for 03-labels.py example during CI documentation build).
  • Updates made in doc/sphinx_gallery_examples/06-plotting/07-plot_on_geometries.py to improve visibility of elements on the plot.

@moe-ad moe-ad self-assigned this Mar 2, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 2, 2026

Codecov Report

❌ Patch coverage is 80.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.19%. Comparing base (30c2026) to head (37fe896).
⚠️ Report is 89 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3023      +/-   ##
==========================================
+ Coverage   82.62%   83.19%   +0.56%     
==========================================
  Files          92       92              
  Lines       11223    11227       +4     
==========================================
+ Hits         9273     9340      +67     
+ Misses       1950     1887      -63     

@moe-ad moe-ad added the deploy-pr-doc For deploying a PR's documentation label Mar 5, 2026
@moe-ad moe-ad changed the title tests: switch plotting tests to viz interface chore: switch default plotting backend to viz interface Mar 6, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 6, 2026

The documentation for this pull request will be available at https://dpf.docs.pyansys.com/pull/3023. Please allow some time for the documentation to be deployed.

@moe-ad moe-ad requested a review from PProfizi March 6, 2026 10:20
Comment thread .github/workflows/tests.yml
Comment thread src/ansys/dpf/core/plotter.py
@moe-ad moe-ad merged commit 0430c64 into main Mar 9, 2026
173 of 178 checks passed
@moe-ad moe-ad deleted the test/switch-tests-to-viz-interface branch March 9, 2026 23:37
@github-actions
Copy link
Copy Markdown
Contributor

This PR has been closed. Documentation for this pull request will shortly be removed from its former deployment address.

@PProfizi PProfizi added the enhancement New feature or request label Apr 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deploy-pr-doc For deploying a PR's documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants