Add a plot header file for sampling particles #1435
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Add a
Header
file for sampling particles. Long story of why is below.Pull request type
Please check the type of change introduced:
Checklist
The following is included:
This PR was tested by running:
Additional background
Issue #1417 brought to our attention that Paraview >= 5.12 could not read our sampling particles any more.
This was reported to ParaView here: https://gitlab.kitware.com/paraview/paraview/-/issues/22835.
It turns out that Paraview in this PR started requiring a Header file that was written by the WritePlotFile routine (independently of the particles). In the majority of amrex applications, particles are written out in conjunction with a plot file (and in the same directory) so this is typically not an issue because the plot file header is available. However, the AMR-Wind sampling particles are independent (can be written at a different frequency, etc) so our particle directories did not have a Header file.
This PR adds a Header file to the sampling particle directories. I can now use ParaView 5.13 to viz our particles. The added bonus is that it now recognizes the time stamp associated to those particles.
Tagging @rthedin as interested.