-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add WOA23 initial condition #610
Conversation
TestingI ran all of the QU240, EC30to60 and ECwISC30to60 test cases for the WOA23 initial condition on Chrysalis. Some of the
and I don't expect that test case to be affected by a different initial condition except for the metadata. The results can be found in these 3 directories:
|
@milenaveneziani and @vanroekel, I'm not sure the best way to proceed here. I guess I'm inclined to have you look at the EC30to60 initial condition:
and E3SM initial condition (after dynamic adjustment):
in whatever way makes sense to you (maybe just ParaVeiw?) and let me know if any issues jump out at you. |
@xylar: yes, I am sorry for not checking in. This is definitely on my radar. I just had a couple of deadlines this week, but I will use paraview to look at the new initial condition hopefully by tomorrow. |
@milenaveneziani, this is different from #604, just so that's clear. #604 is on the original WOA 2023 grid and this is on an MPAS mesh. So I'm not trying to nag you or push you, I just figured it was time to move on from #604 since no one had time to give immediate feedback (which is totally understandable). At least this way we know that what I did in #604 does actually work in MPAS-Ocean and compass as expected, we just need to look at the results and see if they don't look crazy. Thanks for getting to this when you can! |
@xylar the IC name is mpaso.EC30to60E2r3.20230420.nc. does this imply that the grid is different from the standard EC30to60E2r2? I was going to offer to run a case with the new IC, but not if it is a new grid. |
@maltrud the r3 has the same horizontal mesh as the r2 EC30to60E2 but the bathymetry is changed to GEBCO from ETOPO. So you should be able to run pretty easily with it |
@maltrud, this isn't a mesh that we're going to use in E3SM as it is, I'm just testing out the WOA23 initial condition. When it's time to go to E3SM, I'll change the E3SM verison to E3 and the revision number back to r1. Don't worry too much about the mesh name for now. |
Yeah, compass is generating a new jigsaw mesh for these tests so they're not the same as anything we're already using. I don't see a lot of value in running E3SM simulations yet, nor in making a WOA23 initial condition for EC30to60E2r2. We'll run testing later on but I'm just looking for a sanity check for now. |
@vanroekel, yes that's true but (despite the name here), this is not that mesh either. It's a work-in-progress toward EC(wISC)30to60E3r1. |
Oh I see. Thanks for the clarification @xylar |
The confusion is coming from the fact that the |
@xylar and all, is it worthwhile/possible to look at a few vertical T/S profiles in addition to the surface fields? |
Actually, I suppose the way to handle that is to just bump the revision number immediately after we merge a mesh PR, so all subsequent testing has the next revision number until we actually use that number. I can go ahead and do that. |
The paraview extractor can pull out various depth indices. That's what I had in mind. |
But, no, we don't have great tools for making profiles of initial conditions yet. @milenaveneziani, do you have anything up your sleeve? |
Hopefully, #611 will reduce some of the confusion about mesh names in future testing. |
@dengwirda, it should be possible for me to throw together a python script to use the transect plotting from Here's what I'll start from: MPAS-Dev/MPAS-Tools#324 (comment) |
Thanks @xylar, I was considering whether assessing (some approximation to) the stability of the stratification may be a good sanity check on the remapping onto the MPAS layers, and/or might help identify columns where it's not quite right. As you say though, this would require a bunch of infrastructure to be put in place, and might be a bridge too far here. |
@dengwirda, @xylar: yes, I do have scripts that make either a vertical profile across a certain transect (say T as a function of x and z) or vertical profiles for a property averaged over a certain region(T(z)). I can share the scripts if interested. |
Add a `phc` suite to run the PHC versions of test cases from `pr`.
@dengwirda and @milenaveneziani (and anyone else), feel free to chuck transect geojson files my way and I can make more plots. Alternatively you can follow these instructions to try it out yourselves: |
Beautiful figures @xylar, thanks! |
@milenaveneziani, thanks for that. Here are some plots based on your transects and mine for the SORRM initial condition (before dynamic adjustment): I will also post the same after dynamic adjustment: And both of those for the WC14 mesh: Keep an eye on those links. |
will do! plots look great, thanks! I generated the vtk files and now I am trying to transfer them to my laptop for viewing them with paraview. Just having some trouble with ssh and port 22 connections these days.. |
Sea surface height, secure shell or both? ;-) |
ok, I was finally able to look at both the initial_state and after_adjustment file. I don't see anything weird about them (I mostly looked at T and S, various layers, just browsing through paraview). No weird blob is visible in the Arctic or anywhere else. |
There was a problem hiding this 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 new ic files in paraview.
Regarding #610 (comment), I think the extrapolation algorithm is doing exactly what it's meant to do. The weirdness comes from changes in FRIS bathymetry that mean that different inlets get cut off at different depths and the fact that there's a pretty strong salinity gradient from east to west. The following are from ParaVeiw looking at the WOA23 data set on the 0.25 degree mesh at an intermediate stage of the extrapolation. At shallower depths, there are contributions from the western and central Ronne, and Filchner, resulting in a moderate salinity: Then, the central Ronne gets cut off at intermediate depths, so the center of the Ronne cavity suddenly becomes less saline: And then at deeper depths, the western Ronne gets cut off and only the more saline water from Filchner is avaliable: Obviously, this isn't super realistic but I don't see how we're going to do a ton better with just extrapolating. It seems like dynamic adjustment isn't too upset about this density distribution. More as soon as SORRM is finished with dynamic adjustment... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I took a look through a bunch of paraview slices and combined with figures posted by @xylar I don't see anything odd. So am approving this PR.
@vanroekel and @milenaveneziani, thank you so much for the feedback! Anyone else, I certainly invite you to continue taking a look at the results. But I'm going to merge this for now. There will still be time to look over the initial condition and a (more) spun-up E3SM run before we approve the E3SM v3 meshes and initial conditions. |
This merge adds the WOA 2023 climatology from 1991-2020 as the default initial condition in the global ocean test group.
The WOA23 data was processed using #604.
Checklist
Testing
in this PR) any testing that was used to verify the changes