Skip to content

Get Omega reference density from polaris.constants when computing z-tilde#489

Merged
xylar merged 1 commit intoE3SM-Project:mainfrom
xylar:get-omega-rho0-from-pcd
Mar 12, 2026
Merged

Get Omega reference density from polaris.constants when computing z-tilde#489
xylar merged 1 commit intoE3SM-Project:mainfrom
xylar:get-omega-rho0-from-pcd

Conversation

@xylar
Copy link
Copy Markdown
Collaborator

@xylar xylar commented Mar 11, 2026

With this merge, we no longer support a polaris config option for rho0, instead relying on seawater_density_reference that will soon come from the Physical Constants Dictionary (PCD).

This is the Polaris equivalent of E3SM-Project/Omega#358.

Checklist

  • Testing comment in the PR documents testing used to verify the changes

@xylar xylar requested a review from alicebarthel March 11, 2026 10:48
@xylar
Copy link
Copy Markdown
Collaborator Author

xylar commented Mar 11, 2026

@alicebarthel, could you have a quick look when you can? No existing Polaris tests use the functions from which I have removed rho0 as a parameter, but #465 will. I'll rebase that branch onto this one.

With this merge, we no longer support a polaris config option for
rho0, instead relying on `seawater_density_reference` that will
soon come from the Physical Constants Dictionary (PCD).
@xylar xylar force-pushed the get-omega-rho0-from-pcd branch from f785b7f to d99536a Compare March 11, 2026 10:51
@xylar xylar changed the title Get Omega rho0 from polaris.constants in z-tilde Get Omega reference density from polaris.constants when computing z-tilde Mar 11, 2026
@xylar
Copy link
Copy Markdown
Collaborator Author

xylar commented Mar 11, 2026

Testing

Using tests from #465, I'm seeing all passing on Chrysalis with Intel:

0:01:40 PASS ocean/horiz_press_grad/salinity_gradient
0:01:32 PASS ocean/horiz_press_grad/temperature_gradient
0:01:32 PASS ocean/horiz_press_grad/ztilde_gradient
Total runtime: 0:04:45

It doesn't look like the switch from rho0 = 1035 to 1026 made any difference in terms of number of iterations needed to compute the pseudo-thickness initial condition either.

The omega_pr suite also passed on Chrysalis with Intel (even without #488!).

@alicebarthel
Copy link
Copy Markdown
Contributor

@xylar This looks good to me based on visual inspection of the code changes.
Did you need any extra tests on this?

@xylar
Copy link
Copy Markdown
Collaborator Author

xylar commented Mar 12, 2026

@alicebarthel, nope, just wanted a quick visual inspection like you did. Thanks!

@xylar xylar merged commit 9f11d07 into E3SM-Project:main Mar 12, 2026
7 checks passed
@xylar xylar deleted the get-omega-rho0-from-pcd branch March 12, 2026 06:47
@xylar
Copy link
Copy Markdown
Collaborator Author

xylar commented Mar 12, 2026

We missed


Should follow up sometime soon...

@alicebarthel
Copy link
Copy Markdown
Contributor

Good catch, @xylar!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants