Skip to content

Conversation

@anslpa
Copy link
Contributor

@anslpa anslpa commented Jul 25, 2023

This intends to add a "compute_streamline" step to access a Streamlines object before plotting it.
It create an "helpers" module in ansys.dpf.coren contaiing a streamlines module.

@codecov
Copy link

codecov bot commented Jul 25, 2023

Codecov Report

Merging #1062 (7b9e8e2) into plot/streamlines (726c8eb) will decrease coverage by 0.13%.
The diff coverage is 50.00%.

@@                 Coverage Diff                  @@
##           plot/streamlines    #1062      +/-   ##
====================================================
- Coverage             87.95%   87.82%   -0.13%     
====================================================
  Files                    78       80       +2     
  Lines                  9016     9038      +22     
====================================================
+ Hits                   7930     7938       +8     
- Misses                 1086     1100      +14     

@anslpa anslpa merged commit 39d048f into plot/streamlines Jul 25, 2023
@anslpa anslpa deleted the helpers/compute_streamlines branch July 25, 2023 15:05
anslpa added a commit that referenced this pull request Jul 26, 2023
* Add "add_streamlines" methods

* Add plot streamlines example

* field as field only (no list)

* Use download files

* Correctly compute streamlines field name

* Docstring added

* filter kwargs

* Add time report (to revert)

* Update example with better values for elbow

* Revert "Add time report (to revert)"

This reverts commit 55f8ae1.

* Add permissive

* Surface streamline example

* show_plane argument

* Style check

* Style check

* show_plane replaced by plane

* Update docstring plane

* Update examples/06-plotting/08_plot_3d_streamlines.py

Co-authored-by: JennaPaikowsky <[email protected]>

* Update examples/06-plotting/08_plot_3d_streamlines.py

Co-authored-by: JennaPaikowsky <[email protected]>

* Update examples/06-plotting/08_plot_3d_streamlines.py

Co-authored-by: JennaPaikowsky <[email protected]>

* Update examples/06-plotting/08_plot_3d_streamlines.py

Co-authored-by: JennaPaikowsky <[email protected]>

* Update examples/06-plotting/08_plot_3d_streamlines.py

Co-authored-by: JennaPaikowsky <[email protected]>

* Update examples/06-plotting/09_plot_surface_streamlines.py

Co-authored-by: JennaPaikowsky <[email protected]>

* Update examples/06-plotting/09_plot_surface_streamlines.py

Co-authored-by: JennaPaikowsky <[email protected]>

* Update src/ansys/dpf/core/plotter.py

Co-authored-by: JennaPaikowsky <[email protected]>

* Remove

* style check

* Helpers: compute streamlines (#1062)

* Add helpers

* enhance descr

* Prepare "compute_streamlines" method and adapt the examples/plot code. Remaining is the implementation.

* try pyvista import

* protoype working return pv.DataSets and pv.PolyData

* fix raising

* create helpers module as folder

* update src

* use translation methods

* typo

* first tries for creating pvDataSet

* Add streamlines and streamlinessource classes

* Remove meshed_region and field attributes from add_streamline, update all examples to use "compute_streamline"

* replace computed_streamlines and computed_source

* update docstring

* update docstring and fix computed_streamlines

* style check

* style check

* use pv_data_set

* Fix doc

* 12 to 13 (examples folder)

* Streamlines: add base class for Streamlines and StreamlinesSource objects (#1070)

* Implement base class for streamlines object, to prepare the conversion of the _pv_data_set to the _fc

* fix faces

* style check

* style check

* style check

* Update examples/13-streamlines/README.txt

Co-authored-by: Paul Profizi <[email protected]>

* fix comment

* Set permissive default value to True

* Update max_time

---------

Co-authored-by: JennaPaikowsky <[email protected]>
Co-authored-by: Paul Profizi <[email protected]>
@PProfizi PProfizi added the enhancement New feature or request label Aug 2, 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.

3 participants