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

Post refactor - First PR #208

Merged
merged 42 commits into from
Dec 19, 2022
Merged

Post refactor - First PR #208

merged 42 commits into from
Dec 19, 2022

Conversation

germa89
Copy link
Contributor

@germa89 germa89 commented Dec 8, 2022

Adds Solution with solution.py
Adds Selection, TimeFreqSelection, SpatialSelection with selection.py
Adds DataObject with data_object.py
Adds Mesh with mesh.py
Adds tools.select with tools.py

Adds temporary example and meeting notes in temp/

@codecov
Copy link

codecov bot commented Dec 8, 2022

Codecov Report

Merging #208 (9741661) into post_refactor (767d8d2) will decrease coverage by 6.74%.
The diff coverage is 44.91%.

❗ Current head 9741661 differs from pull request most recent head 4c55b06. Consider uploading reports for the commit 4c55b06 to get more accurate results

@@                Coverage Diff                @@
##           post_refactor     #208      +/-   ##
=================================================
- Coverage          83.64%   76.89%   -6.75%     
=================================================
  Files                 25       30       +5     
  Lines               1351     1632     +281     
=================================================
+ Hits                1130     1255     +125     
- Misses               221      377     +156     

@PProfizi PProfizi changed the title Post refactor solution Post refactor - Solution Dec 16, 2022
@PProfizi PProfizi changed the title Post refactor - Solution Post refactor - First PR Dec 16, 2022
Copy link
Contributor

@GuillemBarroso GuillemBarroso left a comment

Choose a reason for hiding this comment

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

LGTM. I just added some minor suggestions.

@PProfizi PProfizi merged commit 9346236 into post_refactor Dec 19, 2022
@PProfizi PProfizi deleted the post_refactor_solution branch December 19, 2022 18:01
PProfizi added a commit that referenced this pull request Mar 8, 2023
* selection implementation

* workflow

* Create a Mesh class

* Work on the displacement method (state before using Selection as a workflow)

* Testing

* Rework TimeFreqSelection and SpatialSelection

* Overwritting call to dpfmechanical object

* Reorg

* Add DataObject. (#210)

* Add DataObject.

* Update src/ansys/dpf/post/data_object.py

* Update src/ansys/dpf/post/data_object.py

* pre meeting

* Add Solution.results property

* Create the PandasImportError

* Fix circular import due to Type Hinting

* fixes for demo

* Set the pydpf-core dependency onto a branch instead of the latest released version

* Remove ssh from core dependency link

* Remove ssh from core dependency link

* Use master branch of pydpf-core and do not import core.geometry_factory yet.

* Fix List[str] typehinting for Python < 3.9

* Fix Optional typehinting unavailable for Python < 3.9

* Fix Optional typehinting unavailable for Python < 3.9

* Fix segfault at import

* Fix pre-commit test_selection.py

* Fix pre-commit data_object.py

* Fix pre-commit errors.py

* Fix pre-commit mesh.py

* Fix pre-commit example.py

* Fix pre-commit meeting.py

* Fix Union

* Fix List for 3.8

* Change back to legacy solution before creating the argument

* Add pandas to requirements_test.txt

* Fix black static_analysis.py

* pandas 1.3.5 is the latest with support for Python 3.7

* Add legacy argument to load_solution, apply it where necessary.

* Fix test_selection.py

* Comment-out tests not yet working. Solve in specific PR

* Apply suggestions from code review

Co-authored-by: Guillem Barroso <[email protected]>

* Revert getting name from __name__

Co-authored-by: cbellot <[email protected]>
Co-authored-by: PProfizi <[email protected]>
Co-authored-by: Maxime Rey <[email protected]>
Co-authored-by: PProfizi <[email protected]>
Co-authored-by: Guillem Barroso <[email protected]>
PProfizi added a commit that referenced this pull request Mar 8, 2023
* selection implementation

* workflow

* Create a Mesh class

* Work on the displacement method (state before using Selection as a workflow)

* Testing

* Rework TimeFreqSelection and SpatialSelection

* Overwritting call to dpfmechanical object

* Reorg

* Add DataObject. (#210)

* Add DataObject.

* Update src/ansys/dpf/post/data_object.py

* Update src/ansys/dpf/post/data_object.py

* pre meeting

* Add Solution.results property

* Create the PandasImportError

* Fix circular import due to Type Hinting

* fixes for demo

* Set the pydpf-core dependency onto a branch instead of the latest released version

* Remove ssh from core dependency link

* Remove ssh from core dependency link

* Use master branch of pydpf-core and do not import core.geometry_factory yet.

* Fix List[str] typehinting for Python < 3.9

* Fix Optional typehinting unavailable for Python < 3.9

* Fix Optional typehinting unavailable for Python < 3.9

* Fix segfault at import

* Fix pre-commit test_selection.py

* Fix pre-commit data_object.py

* Fix pre-commit errors.py

* Fix pre-commit mesh.py

* Fix pre-commit example.py

* Fix pre-commit meeting.py

* Fix Union

* Fix List for 3.8

* Change back to legacy solution before creating the argument

* Add pandas to requirements_test.txt

* Fix black static_analysis.py

* pandas 1.3.5 is the latest with support for Python 3.7

* Add legacy argument to load_solution, apply it where necessary.

* Fix test_selection.py

* Comment-out tests not yet working. Solve in specific PR

* Apply suggestions from code review

Co-authored-by: Guillem Barroso <[email protected]>

* Revert getting name from __name__

Co-authored-by: cbellot <[email protected]>
Co-authored-by: PProfizi <[email protected]>
Co-authored-by: Maxime Rey <[email protected]>
Co-authored-by: PProfizi <[email protected]>
Co-authored-by: Guillem Barroso <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants