Skip to content
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

Change SORRMv3 ocean initial condition #6758

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

darincomeau
Copy link
Member

@darincomeau darincomeau commented Nov 19, 2024

Changes ocean initial condition for B-cases with the SOwISC12to30E3r3 ocean mesh. The previous default initial condition is from a one-month G-case restart from cold-start (standard default). The initial condition here has been interpolated from year 701 of the SORRM v2.1 1950 control simulation, in an effort to bypass/speed-up ocean spinup.

New ocean initial condition file is staged in public inputdata repo, world-readable.

[BFB] except
[non-BFB] for B-cases with the SOwISC12to30E3r3 ocean mesh (not in current testing).

@darincomeau
Copy link
Member Author

@proteanplanet
Copy link
Contributor

I'd like to request @maltrud document on this PR, however briefly, the method used to obtain these results, so that we have that information clustered with this application.

@maltrud
Copy link
Contributor

maltrud commented Nov 21, 2024

will do ASAP

@rljacob
Copy link
Member

rljacob commented Dec 19, 2024

@maltrud still waiting for docs.

1 similar comment
@rljacob
Copy link
Member

rljacob commented Jan 9, 2025

@maltrud still waiting for docs.

@rljacob
Copy link
Member

rljacob commented Jan 24, 2025

@maltrud still waiting for you to provide requested docs.

@maltrud
Copy link
Contributor

maltrud commented Jan 24, 2025

here is an outline of how I generated the initial condition.
creation_of_SOwISCE3r3_initial_condition.pdf

@xylar
Copy link
Contributor

xylar commented Jan 25, 2025

Thanks @maltrud for that document. It's really helpful and the procedure makes sense.

There's one part you don't mention that I'd appreciate having you add. How was the highly non-z-level vertical coordinate handled near and under ice shelves in both meshes? Did you treat it as if it were z-level? Or did it get some special treatment?

@cbegeman
Copy link
Contributor

Thanks, @maltrud! It all was very clear. I noticed that you said the vertical interpolation was linear; was the horizontal interpolation also (bi)linear?

@maltrud
Copy link
Contributor

maltrud commented Jan 27, 2025

@cbegeman and @xylar thanks for your questions. I did nothing special to handle ice shelf cavities, just did "horizontal" extrapolation based on vertical index. my plan was to see if this initial method caused problems, and if necessary to modify the procedure to handle cavities, but it seems to have worked ok without special treatment. and, yes, the horizontal interpolation was bilinear. I'll add this info to the document.

@xylar
Copy link
Contributor

xylar commented Jan 27, 2025

@maltrud, okay, I'm fine with that. While the vertical grids under ice shelves are pretty unrelated to the z-level grids in the open ocean (and thus the interpolation won't necessarily be very accurate), the flushing time for cavities is probably on the order of a few years even for the largest cavities so these differences shouldn't have a lasting impact.

@rljacob
Copy link
Member

rljacob commented Jan 28, 2025

@cbegeman or @proteanplanet this still needs an approval from one of you.

Copy link
Contributor

@cbegeman cbegeman left a comment

Choose a reason for hiding this comment

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

Approving on the basis of @darincomeau's simulations, particularly the smaller drift in OHC, as well as @maltrud's methodological justification. Thanks @maltrud for all your work generating this IC!

@maltrud
Copy link
Contributor

maltrud commented Jan 28, 2025

updated document that describes the creation the new initial condition which incorporates answers to the questions raised by @cbegeman and @xylar .
creation_of_SOwISCE3r3_initial_condition.pdf

Copy link
Contributor

@proteanplanet proteanplanet left a comment

Choose a reason for hiding this comment

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

Thank you for the documentation @maltrud and for the simulations @darincomeau proving the utility.

jonbob added a commit that referenced this pull request Jan 29, 2025
Change SORRMv3 ocean initial condition

Changes ocean initial condition for B-cases with the SOwISC12to30E3r3
ocean mesh. The previous default initial condition is from a one-month
G-case restart from cold-start (standard default). The initial condition
here has been interpolated from year 701 of the SORRM v2.1 1950 control
simulation, in an effort to bypass/speed-up ocean spinup.

New ocean initial condition file is staged in public inputdata repo,
world-readable.

[BFB] for all currently tested configurations
@jonbob
Copy link
Contributor

jonbob commented Jan 29, 2025

Passes:

  • SMS_D_Ld1.ne30pg2_r05_IcoswISC30E3r5.WCYCL1850.chrysalis_intel.allactive-wcprod

merged to next

@jonbob jonbob merged commit 8b5891b into E3SM-Project:master Jan 30, 2025
12 checks passed
@jonbob
Copy link
Contributor

jonbob commented Jan 30, 2025

merged to master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BFB PR leaves answers BFB mpas-ocean
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants