Skip to content

add an analysis script for the MHD LoopAdvection problem#3341

Open
zingale wants to merge 1 commit intoAMReX-Astro:developmentfrom
zingale:add_mhd_vis
Open

add an analysis script for the MHD LoopAdvection problem#3341
zingale wants to merge 1 commit intoAMReX-Astro:developmentfrom
zingale:add_mhd_vis

Conversation

@zingale
Copy link
Copy Markdown
Member

@zingale zingale commented May 2, 2026

PR summary

PR motivation

PR checklist

  • test suite needs to be run on this PR
  • this PR will change answers in the test suite to more than roundoff level
  • all newly-added functions have docstrings as per the coding conventions
  • the CHANGES file has been updated, if appropriate
  • if appropriate, this change is described in the docs

@zingale zingale added the mhd label May 2, 2026
Copy link
Copy Markdown
Member

@guadabsb15 guadabsb15 left a comment

Choose a reason for hiding this comment

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

Looks very similar to what I had. The only extra thing I had was something like sp.annotate_timestamp() before saving. But it is totally optional, It was just easier for me to show then the plots with the time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants