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

Land surface upgrades for HR4 #2354

Merged
merged 19 commits into from
Jul 24, 2024

Conversation

HelinWei-NOAA
Copy link
Collaborator

@HelinWei-NOAA HelinWei-NOAA commented Jul 2, 2024

Commit Queue Requirements:

  • Fill out all sections of this template.
  • All sub component pull requests have been reviewed by their code managers.
  • Run the full Intel+GNU RT suite (compared to current baselines) on either Hera/Derecho/Hercules
  • Commit 'test_changes.list' from previous step

Description:

To address some concerns from HR3 especially the surface cold biases we have finalized some land surface upgrades after carrying out extensive tests and evaluation.

  • remove vegetation type-dependent for the diagnostic option (opt_diag) used in HR3
  • use radiative temperature as skin temperature
  • use single loop alternative approach to address some flux inconsistence below and above the canopy
  • fix the thermal roughness bug (the one for bared portion in the composition is not the same as the one used in the bared title)
  • adjust some momentum roughness length
  • use the interpolated temp between the skin temp and the first soil temp to get the initial snow temp based on snow depth
  • use constant snow thermal conductivity to reduce the snow insulation effects
  • fix radiation parameter bugs for grassland/cropland
  • adjust albedo parameter for each soil color category

Commit Message:

* UFSWM - 
 
  * FV3 - 
    * ccpp-physics - 

  * NOAHMP - 

Priority:

  • Normal.

Git Tracking

UFSWM:

  • Closes #
  • None

Sub component Pull Requests:

Changes

Regression Test Changes (Please commit test_changes.list):

this PR creates new baselines, changes baselines

Please make sure you have properly submitted test_changes.list

  • PR Adds New Tests/Baselines.
  • PR Updates/Changes Baselines.

Testing Log:

  • RDHPCS
    • Hera
    • Orion
    • Hercules
    • Jet
    • Gaea
    • Derecho
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
  • opnReqTest (complete task if unnecessary)

@yangfanglin
Copy link
Collaborator

@uturuncoglu do you need to update the land component model for this upgrade ?

@uturuncoglu
Copy link
Collaborator

@yangfanglin I think this PR also updates the component model right. Please double check that both component model and ccpp/physics have same version of code. As I see this PR makes some variables optional, I am not sure how that needs to be handled in the component model but if the RTs related with component model passes without any issue, I think it is good to know.

@jkbk2004
Copy link
Collaborator

@HelinWei-NOAA can you sync up branches? we hope to get this pr ready to commit.

@jkbk2004
Copy link
Collaborator

@yangfanglin feel free to review and approve NOAA-EMC/noahmp#11. We will do pre-test to make sure.

@HelinWei-NOAA
Copy link
Collaborator Author

@jkbk2004 Done

@HelinWei-NOAA can you sync up branches? we hope to get this pr ready to commit.

@FernandoAndrade-NOAA FernandoAndrade-NOAA added Baseline Updates Current baselines will be updated. Waiting for Reviews The PR is waiting for reviews from associated component PR's. labels Jul 22, 2024
@FernandoAndrade-NOAA
Copy link
Collaborator

FernandoAndrade-NOAA commented Jul 22, 2024

It seems the NOAHMP sub PR was merged early. In that case I'd recommend the commit ID be synced again and we continue with testing. If there's an issue, we will need to roll back the changes from the early merge in the subcomponent. The FV3 sub PR also still needs an initial approval.

@BrianCurtis-NOAA @zach1221

@FernandoAndrade-NOAA
Copy link
Collaborator

FernandoAndrade-NOAA commented Jul 22, 2024

@HelinWei-NOAA please go ahead and revert .gitmodules NOAHMP section and update the hash, it should now be 3ac32f0
Please also be sure to fill out the commit message section of the PR description.

@HelinWei-NOAA
Copy link
Collaborator Author

@HelinWei-NOAA please go ahead and revert .gitmodules NOAHMP section and update the hash, it should now be 3ac32f0 Please also be sure to fill out the commit message section of the PR description.

@FernandoAndrade-NOAA Done

@FernandoAndrade-NOAA FernandoAndrade-NOAA added Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. and removed Waiting for Reviews The PR is waiting for reviews from associated component PR's. labels Jul 22, 2024
@BrianCurtis-NOAA
Copy link
Collaborator

@HelinWei-NOAA please make sure to update what you want for the commit message in the top section. It will be used once we get to the merge stage.

@zach1221 zach1221 added jenkins-ort run ORT testing and removed jenkins-ort run ORT testing labels Jul 22, 2024
@FernandoAndrade-NOAA
Copy link
Collaborator

Acorn will be skipped this PR, I'll let CCPP to continue with the merge process.

@zach1221
Copy link
Collaborator

@HelinWei-NOAA please let us know what commit message you'd like for this PR.

@HelinWei-NOAA
Copy link
Collaborator Author

HelinWei-NOAA commented Jul 24, 2024

the land surface upgrades for HR4

@HelinWei-NOAA please let us know what commit message you'd like for this PR.

@jkbk2004 jkbk2004 merged commit 7544ee8 into ufs-community:develop Jul 24, 2024
3 checks passed
@FernandoAndrade-NOAA FernandoAndrade-NOAA mentioned this pull request Jul 24, 2024
14 tasks
dpsarmie added a commit to dpsarmie/ufs-weather-model that referenced this pull request Jul 24, 2024
With LSM upgrades in PR ufs-community#2354, the IOPT_DIAG option will
now be set to 2 instead of 1. This change was in most
RTs. This commit will change the option for the remaining
RTs that did not have this option.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Baseline Updates Current baselines will be updated. Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants