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

rename {x,y,z}-velocity to {x,y,z}-RiemannSolverVelocity in plotfiles #818

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

BenWibking
Copy link
Collaborator

@BenWibking BenWibking commented Dec 12, 2024

Description

Renames the plotfile variables for the cell-centered averages of the face-centered velocities from the Riemann solver to: x-RiemannSolverVelocity, y-RiemannSolverVelocity, and z-RiemannSolverVelocity.

Related issues

N/A

Checklist

Before this pull request can be reviewed, all of these tasks should be completed. Denote completed tasks with an x inside the square brackets [ ] in the Markdown source below:

  • I have added a description (see above).
  • I have added a link to any related issues (if applicable; see above).
  • I have read the Contributing Guide.
  • I have added tests for any new physics that this PR adds to the code.
  • (For quokka-astro org members) I have manually triggered the GPU tests with the magic comment /azp run.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Dec 12, 2024
@BenWibking
Copy link
Collaborator Author

/azp run

Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@BenWibking BenWibking changed the title rename {x,y,z}-velocity to {x,y,z}-RiemannSolverVelocity rename {x,y,z}-velocity to {x,y,z}-RiemannSolverVelocity in plotfiles Dec 12, 2024
@BenWibking BenWibking enabled auto-merge December 12, 2024 21:44
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 12, 2024
@BenWibking BenWibking added this pull request to the merge queue Dec 12, 2024
Merged via the queue into development with commit a4a1d46 Dec 12, 2024
23 checks passed
@BenWibking BenWibking deleted the BenWibking/rename-fc-velocity branch December 14, 2024 03:15
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants