Skip to content

Revise z_bot and p_bot calculation to avoid negative z_bot #102

Merged
laurenchilutti merged 1 commit into
NOAA-GFDL:dev/emcfrom
XiaqiongZhou-NOAA:zbot
May 12, 2021
Merged

Revise z_bot and p_bot calculation to avoid negative z_bot #102
laurenchilutti merged 1 commit into
NOAA-GFDL:dev/emcfrom
XiaqiongZhou-NOAA:zbot

Conversation

@XiaqiongZhou-NOAA
Copy link
Copy Markdown
Contributor

Description
The calculation of z_bot and p_bot is revised to avoid negative z_bot.
The calculation of the lowest layer-mean hydrostatic pressure p_lot is revised by only using pe (interface values) only instead of delp and peln in order to avoid the inconsistency between them. z_bot is now exactly the height depth from the surface to the lowest model level. This modification can fix the model crash issue for UFS coupled applications.

How Has This Been Tested?

This was tested on Hera and Orion for the coupled regression tests.
The z_bot fields are more reasonable with the modification.

Checklist:

Please check all whether they apply or not

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules

Copy link
Copy Markdown
Contributor

@lharris4 lharris4 left a comment

Choose a reason for hiding this comment

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

looks fine

@HenryJuang-NOAA
Copy link
Copy Markdown

HenryJuang-NOAA commented May 10, 2021 via email

@bensonr
Copy link
Copy Markdown
Contributor

bensonr commented May 10, 2021

@HenryJuang-NOAA - after you click on green "add your review" button near the top of this page, there green button on the right hand side of the screen near the top that says "review changes". It is there you can click approve and submit your review.

Copy link
Copy Markdown

@HenryJuang-NOAA HenryJuang-NOAA left a comment

Choose a reason for hiding this comment

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

Is bot for T_bot and P_bot to indicate T and P at the bottom first model layer location, then z_bot must mean height at the bottom model layer location. Based on the formula, it is okay to say the depth in height above the model ground or ocean surface.

Copy link
Copy Markdown
Contributor

@bensonr bensonr left a comment

Choose a reason for hiding this comment

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

I agree with @HenryJuang-NOAA regarding the specifications of where the data originates, but won't hold up the approval. The inline documentation should match up to the quantities provided.

@laurenchilutti laurenchilutti merged commit 3720aea into NOAA-GFDL:dev/emc May 12, 2021
@XiaqiongZhou-NOAA XiaqiongZhou-NOAA deleted the zbot branch March 29, 2023 16:45
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.

6 participants