Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Improvements to plotting function for flow-over-heated-plate #570

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

j-signorelli
Copy link
Contributor

This PR updates and improves the plot-final-interface-temperature.py file:

  • Previously, the file would only run properly when all solid solvers were run; now it works regardless of how many have been run
  • When data was not found, the last file failed because data_id was not defined, and if not data_id would cause an error. This updated script correctly prints what files/data is/are not found in the search.
  • General improvements (such as using enumerate, removing unnecessary retrieval of other data arrays, etc.)
  • Because either SU2 or OpenFOAM may be used as the fluid-participant, the labels were changed as OpenFOAM-nutils to Fluid-nutils for example.

This PR also adds SU2 to the metadata.yaml file in flow-over-heated-plate.

No user-facing changes.

Checklist:

  • I added a summary of any user-facing changes (compared to the last release) in the changelog-entries/<PRnumber>.md.
  • I will remember to squash-and-merge, providing a useful summary of the changes of this PR.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant