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

Replacing post_processing and result with DPF #2

Closed
4 tasks
germa89 opened this issue Feb 22, 2022 · 2 comments
Closed
4 tasks

Replacing post_processing and result with DPF #2

germa89 opened this issue Feb 22, 2022 · 2 comments
Assignees
Labels
Client Enhancement New feature or request

Comments

@germa89
Copy link
Contributor

germa89 commented Feb 22, 2022

This is going to be tracked here: ansys/pymapdl#1298

Old issue

Summary

Currently there is duplication regarding the post-processing of MAPDL results. There are three libraries which can do this job:

Intended Outcome

Effort should be merged toward only one single library for post-processing. The current general agreement is to go with DPF.

Any post-processing reference and function inside pymapdl should point to DPF equivalent.

How will it work?

Retro-compatibility should be ensured.
Use DPF to implement post_processing module. It requires to change 3-4 functions which are doing most of the heavy work.

Stages/steps

  • Prepare example with DPF for me to get familiar with the library. Using DPF in example pymapdl#1279
  • Use DPF for the mesh module in PyMAPDL. Almost in progress...
    So far, we are pending on the VTK parser. That is key as mentioned in Mesh objects from db files pydpf-core#331
  • Post-processing - It does not need to be migrated to DPF because it only uses the reader for the mesh, which should be fixed in the previous bullet point.
  • Result. This is going to be tricky.
@germa89 germa89 added Enhancement New feature or request Client labels Feb 22, 2022
@germa89 germa89 self-assigned this Feb 22, 2022
@akaszynski
Copy link

We’re going to need help from the DPF team. @rlagha, can we get help from the new hire?

@rlagha
Copy link

rlagha commented Feb 22, 2022

@akaszynski yes definitely, this is important

@germa89 germa89 changed the title Replacing post_processing and result by DPF Replacing post_processing and result with DPF Jul 27, 2022
@germa89 germa89 closed this as completed Jul 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Client Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants