Skip to content

Clarify registry descriptions for avgSurface* variables#7472

Merged
jonbob merged 2 commits intoE3SM-Project:masterfrom
cbegeman:ocn/fix-variable-descriptions
Jul 25, 2025
Merged

Clarify registry descriptions for avgSurface* variables#7472
jonbob merged 2 commits intoE3SM-Project:masterfrom
cbegeman:ocn/fix-variable-descriptions

Conversation

@cbegeman
Copy link
Copy Markdown
Contributor

Surface values sent to the coupler are not actually extrapolated from zMid to zTop as described. This PR changes the description of these variables in the registry to make that more clear.

[BFB] as there are no changes to the code

cbegeman added 2 commits June 25, 2025 15:59
Surface values sent to the coupler are not actually extrapolated from zMid to zTop as described
@cbegeman cbegeman added MPAS-ocean Concerning the MPAS-ocean model coupled to E3SM. BFB PR leaves answers BFB labels Jun 30, 2025
@cbegeman
Copy link
Copy Markdown
Contributor Author

@vanroekel and @proteanplanet This should be a quick review from you both. I'm mostly bringing it to your attention in case you wanted to comment about the appropriateness of assuming that the surface value of tracers is the zMid value.

Copy link
Copy Markdown
Contributor

@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.

Looks good to me. This is a nice clarification @cbegeman. Thanks!

jonbob added a commit that referenced this pull request Jul 24, 2025
…7472)

Clarify registry descriptions for avgSurface* variables

Surface values sent to the coupler are not actually extrapolated from
zMid to zTop as described. This PR changes the description of these
variables in the registry to make that more clear.

[BFB]
@jonbob
Copy link
Copy Markdown
Contributor

jonbob commented Jul 24, 2025

passes sanity testing, merged to next

@jonbob jonbob merged commit 5cb56bb into E3SM-Project:master Jul 25, 2025
3 checks passed
@jonbob
Copy link
Copy Markdown
Contributor

jonbob commented Jul 25, 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 Concerning the MPAS-ocean model coupled to E3SM.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants