Skip to content

Conversation

@GuillemBarroso
Copy link
Contributor

Fix #712.

This PR improves the plotting capabilities of the .plot() method of the geometric objects Points, Line and Plane.

Changes proposed:

  • geometry.py: accept optional argument mesh so the user can plot the geometric object together with the 3D mesh from the model.
  • plotter.py: accept **kwargs as arguments so pyvista options are accepted.
  • test_geometry.py: add test for plotting geometries together with the mesh.
  • 07-plot_on_geometries.py: improve example also showing how to plot the geometries together with the mesh.

This PR still has to include the changes after #714 is merged.

@GuillemBarroso GuillemBarroso self-assigned this Dec 20, 2022
@codecov
Copy link

codecov bot commented Dec 20, 2022

Codecov Report

Merging #715 (4948cd5) into master (dd35d8c) will increase coverage by 1.45%.
The diff coverage is 97.43%.

@@            Coverage Diff             @@
##           master     #715      +/-   ##
==========================================
+ Coverage   87.17%   88.62%   +1.45%     
==========================================
  Files          71       71              
  Lines        8070     8081      +11     
==========================================
+ Hits         7035     7162     +127     
+ Misses       1035      919     -116     

@GuillemBarroso GuillemBarroso marked this pull request as ready for review December 21, 2022 14:34
@GuillemBarroso GuillemBarroso merged commit 1bc0fea into master Dec 21, 2022
@GuillemBarroso GuillemBarroso deleted the feat/geometries-plotting branch December 21, 2022 16:24
@PProfizi PProfizi added the enhancement New feature or request label Feb 23, 2023
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.

Improve plotting for geometric entities

2 participants