Skip to content

Add FRIS comparison grid#1071

Merged
xylar merged 2 commits into
MPAS-Dev:developfrom
irenavankova:add_fris_comparison_grids
Mar 5, 2025
Merged

Add FRIS comparison grid#1071
xylar merged 2 commits into
MPAS-Dev:developfrom
irenavankova:add_fris_comparison_grids

Conversation

@irenavankova

Copy link
Copy Markdown
Collaborator

Adds the option to make plots on a grid that focuses on the Filchner-Ronne cavity.

Here is an example bsf plot with that option:
barotropicStreamfunction_fris_S12_fris_orig_ANN_years0020-0020

@xylar xylar self-requested a review March 4, 2025 20:51
@xylar xylar self-assigned this Mar 4, 2025
@xylar

xylar commented Mar 4, 2025

Copy link
Copy Markdown
Collaborator

This looks good! I'll give it a couple of quick tests tomorrow and then merge if things work as expected.

@xylar

xylar commented Mar 4, 2025

Copy link
Copy Markdown
Collaborator

Regarding documentation, https://github.com/MPAS-Dev/MPAS-Analysis/blob/develop/docs/users_guide/config/comparison_grids.rst is so woefully out of date that it doesn't make sense to try to fix it here.

@xylar

xylar commented Mar 5, 2025

Copy link
Copy Markdown
Collaborator

I ran the test suite and things look good:
https://web.lcrc.anl.gov/public/e3sm/diagnostic_output/ac.xasay-davis/analysis_testing/chrysalis/add_fris_comparison_grids/

I pushed a commit that adds the Fris comparison grid to the test suite. @irenavankova, I hope that's fine with you.

@xylar xylar merged commit 402df25 into MPAS-Dev:develop Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants