Skip to content

Add WCwISC14 and ARRMwISC12to60 meshes#575

Merged
xylar merged 7 commits intoMPAS-Dev:mainfrom
xylar:add_wcwisc
Mar 28, 2023
Merged

Add WCwISC14 and ARRMwISC12to60 meshes#575
xylar merged 7 commits intoMPAS-Dev:mainfrom
xylar:add_wcwisc

Conversation

@xylar
Copy link
Collaborator

@xylar xylar commented Mar 23, 2023

Checklist

  • User's Guide has been updated
  • Developer's Guide has been updated
  • Documentation has been built locally and changes look as expected
  • Document (in a comment titled Testing in this PR) any testing that was used to verify the changes
  • New tests have been added to a test suite

@xylar xylar added enhancement New feature or request ocean labels Mar 23, 2023
@xylar xylar self-assigned this Mar 23, 2023
@xylar
Copy link
Collaborator Author

xylar commented Mar 23, 2023

Testing

I ran both meshes through all steps but files_for_e3sm, see results in:

/lcrc/group/e3sm/ac.xylar/compass_1.2/chrysalis/test_20230323/arrmwisc
/lcrc/group/e3sm/ac.xylar/compass_1.2/chrysalis/test_20230323/wcwisc14

@xylar
Copy link
Collaborator Author

xylar commented Mar 23, 2023

@milenaveneziani and @vanroekel, these are not even close to the E3SM v3 versions of these meshes yet, I'm just adding the support for creating versions of each mesh with ice-shelf cavities and making sure they "work". Once my test are done (hopefully tomorrow), I'll ask you to have a quick glance at the meshes and any other pieces of the compass process you want. This doesn't have to be a super thorough review since we'll add some more things based on #565 before we do more thorough testing.

@xylar
Copy link
Collaborator Author

xylar commented Mar 24, 2023

@milenaveneziani, the ARRMwISC12to60 is ready to review. While the files_for_e3sm step hasn't been run, I don't think it matters since this isn't the production version of the mesh anyway. I just want to make sure things look okay.

I'd suggest looking at the initial condition (maybe using the paraview extractor):

/lcrc/group/e3sm/ac.xylar/compass_1.2/chrysalis/test_20230323/arrmwisc/ocean/global_ocean/ARRMwISC10to60/PHC/init/initial_state

I mostly just want your sense that nothing looks crazy.

@xylar
Copy link
Collaborator Author

xylar commented Mar 24, 2023

@vanroekel, same for you for the WCwISC14. The files_for_e3sm step started but didn't complete (because making balanced sea-ice partition files is pretty time consuming). I don't think it matters since this isn't the production version of the mesh anyway. I just want to make sure things look okay.

Again, I'd suggest looking at the initial condition (maybe using the paraview extractor):

/lcrc/group/e3sm/ac.xylar/compass_1.2/chrysalis/test_20230323/wcwisc14/ocean/global_ocean/WCwISC14/PHC/init/initial_state

I mostly just want your sense that nothing looks crazy.

@xylar xylar mentioned this pull request Mar 24, 2023
6 tasks
@vanroekel
Copy link
Collaborator

@xylar I took a look through that file you pointed me at in paraview and everything looks reasonable to me. I can see the WC14 enhancement, T&S appear in bounds throughout the column and can see the ISC regions in the south. Let me know if you want me to dump in any paraview screenshots, would be happy to do so.

@xylar
Copy link
Collaborator Author

xylar commented Mar 24, 2023

Thanks @vanroekel! I don't think there's any need to post ParaVeiw screenshots. We'll do a more thorough review later. Thanks for having a look!

xylar added 7 commits March 28, 2023 15:15
Also correct some missing information on test suites and
the SO12to60 mesh.
This is used in SSH adjustment and performance testing, so it
needs to work before dynamic adjustment.
This is needed for ARRM because we shortened the default time
step for short runs (SSH adjustment and performance) and it is
a good idea to provide the time step for WC in case we have to
shorten the default time step for that mesh, too.
@xylar
Copy link
Collaborator Author

xylar commented Mar 28, 2023

@vanroekel, would you be willing to "officially" approve the PR based on the testing you've already done?

Copy link
Collaborator

@vanroekel vanroekel left a comment

Choose a reason for hiding this comment

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

Approving based on thorough visual inspection of the produced initial condition. The resolution is as expected, ice cavities are visible and temperature and salinity are as expected through the full depth of the ocean.

@milenaveneziani
Copy link

I checked the initial state for ARRMwISC10to60 with paraview and everything looks as expected (I also compared bottomDepth to the bathymetry for the ARRM mesh without ice cavities, and they are similar). I checked botttomDepth, layerthickness, temperature, and salinity. Nothing out of the ordinary.
Ice cavities are there where they should be.

@milenaveneziani
Copy link

One thing I wanted to report for record keeping that has nothing to do with this particular mesh, but likely with the PHC data that we use to produce the initial condition, is this small warm pool in the Northwestern Passages just north of Hudson Bay in the middle of winter:

Screen Shot 2023-03-28 at 9 37 56 AM

Just something to be aware of, and pinging @proteanplanet here as well.

Copy link

@milenaveneziani milenaveneziani left a comment

Choose a reason for hiding this comment

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

Approved based on viewing of the initial_state file.

@xylar
Copy link
Collaborator Author

xylar commented Mar 28, 2023

@milenaveneziani, that warm blob is weird! I think the initial condition is a climatology so no particular season. But it certainly suggests a weird feature of the data that we should look into.

@xylar xylar merged commit 9045bdc into MPAS-Dev:main Mar 28, 2023
@xylar xylar deleted the add_wcwisc branch March 28, 2023 15:53
@dengwirda
Copy link
Contributor

@milenaveneziani, I've seen that warm blob in lots of config.'s I believe, and assume it's coming from the underlying data/interpolation somehow.
It doesn't look to me that the raw PHC SST climatology (assuming this is the right one) has the same feature though, which feels a bit weird...
I assume this only affects the spin-up, but starting from a better climatology seems a good thing to improve.

@milenaveneziani
Copy link

Yes, I agree @dengwirda. Best would be to take a look at the PHC files that compass uses to generate the initial condition. The blob is not only warm but also fresh btw. @xylar: could you point me to the original PHC files?

We do use WOA18 climatologies in MPAS-Analysis, so we do have that data ready, I think, if we decide to give it a try.

@xylar
Copy link
Collaborator Author

xylar commented Mar 30, 2023

@xylar: could you point me to the original PHC files?

@milenaveneziani, here are the files we use. @maltrud produced them as far as I'm aware so he might have some thoughts.

https://web.lcrc.anl.gov/public/e3sm/mpas_standalonedata/mpas-ocean/initial_condition_database/PotentialTemperature.01.filled.60levels.PHC.151106.nc
https://web.lcrc.anl.gov/public/e3sm/mpas_standalonedata/mpas-ocean/initial_condition_database/Salinity.01.filled.60levels.PHC.151106.nc

@milenaveneziani
Copy link

Thanks @xylar. I can confirm that the warm/fresh blob is present in the original file that you pointed me to. I wonder if it's PHC itself that has it, since I looked at this other file in the same directory: PotentialTemperature.100levels.Levitus.EN4_1900estimate.200813.nc, and that does not have the blob.

@maltrud
Copy link

maltrud commented Mar 30, 2023

I'll have a look at the original vs filled files.

@maltrud
Copy link

maltrud commented Mar 30, 2023

that weird blob is in the original PHC2.0 data. I didn't realize this was still being used (it's from 15 years ago!). I switched to PHC 3.0 when it came out. I'd say we should switch to WOA (2018?).
Screen Shot 2023-03-30 at 11 33 32 AM

@milenaveneziani
Copy link

I thought that might be the case. Agreed about switching to WOA18.

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

Labels

enhancement New feature or request ocean

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants