Skip to content

rename test_nse_interp -> nse_interp_check + cleaning - #2091

Open
zingale wants to merge 5 commits into
AMReX-Astro:developmentfrom
zingale:nse_interp_fixes
Open

zingale wants to merge 5 commits into
AMReX-Astro:developmentfrom
zingale:nse_interp_fixes

Conversation

@zingale

@zingale zingale commented Sep 12, 2026

Copy link
Copy Markdown
Member

this is consistent with our naming convention that test_* tests use
a grid of cells.

this also reformats the output to be more readable

finally, this adds a test where cubic interpolation violates monotonicity

this is consistent with our naming convention that test_* tests use
a grid of cells.
@zingale

zingale commented Sep 12, 2026

Copy link
Copy Markdown
Member Author

adds the test problem from #2072

@zingale zingale added the nse label Sep 12, 2026
@zhichen3

Copy link
Copy Markdown
Collaborator

the CI yml file can also be renamed for consistency

@zingale

zingale commented Sep 14, 2026

Copy link
Copy Markdown
Member Author

renamed

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