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

Fix/returning only values for selected elements in post processing #777

Conversation

germa89
Copy link
Collaborator

@germa89 germa89 commented Dec 10, 2021

Fix #743 by:

  • Added nodal_values which is used heavily in the post_processing module instead of _ndof_rst. nodal_values returns only the values of the selected nodes.
  • Improved _plot_point_scalars to account for when all the nodes are not selected.
  • There has been some reorg in the functions order.
  • Added two test cases.

…ule instead of `_ndof_rst`.

`nodal_values` returns only the values of the selected nodes.
Improved `_plot_point_scalars` to account for when all the nodes are not selected.
There has been some reorg in the functions order.
…ule instead of `_ndof_rst`.

`nodal_values` returns only the values of the selected nodes.
Improved `_plot_point_scalars` to account for when all the nodes are not selected.
There has been some reorg in the functions order.
…ule instead of `_ndof_rst`.

`nodal_values` returns only the values of the selected nodes.
Improved `_plot_point_scalars` to account for when all the nodes are not selected.
There has been some reorg in the functions order.
@germa89 germa89 self-assigned this Dec 10, 2021
Copy link
Collaborator

@akaszynski akaszynski left a comment

Choose a reason for hiding this comment

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

Thanks for adding this. I'm merging, please see comments.

@akaszynski akaszynski merged commit 4b0f98f into main Dec 13, 2021
@akaszynski akaszynski deleted the fix/returning-only-values-for-selected-elements-in-post_processing branch December 13, 2021 04:44
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.

Size of post_processing output does not match the number of selected elements.
2 participants