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

P7a: Update BM ICs for soil variables release #625

Merged

Conversation

JessicaMeixner-NOAA
Copy link
Collaborator

PR Checklist

  • Ths PR is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR. Please consult the ufs-weather-model wiki if you are unsure how to do this.

  • This PR has been tested using a branch which is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR

  • An Issue describing the work contained in this PR has been created either in the subcomponent(s) or in the ufs-weather-model. The Issue should be created in the repository that is most relevant to the changes in contained in the PR. The Issue and the dependent sub-component PR
    are specified below.

  • If new or updated input data is required by this PR, it is clearly stated in the text of the PR.

Description

This updates the input data date. The update in the input file is so that the land-spin up for NOAH-MP is included in the sfc files for soil variables for the BM7_ICs.

Issue(s) addressed

Testing

Currently, the 35 day tests are being run with a 20 min wall clock to ensure that all IC dates for benchmarks initialize.
Will update the thread when these have completed.

  • hera.intel
  • hera.gnu
  • orion.intel
  • cheyenne.intel
  • cheyenne.gnu
  • gaea.intel
  • jet.intel
  • wcoss_cray
  • wcoss_dell_p3

Dependencies

No dependencies

@JessicaMeixner-NOAA
Copy link
Collaborator Author

The test to make sure the model started with the new sfc* files for the 35 day benchmark runs are completed. Note these only ran for a 20 minute wall clock and were just intended to make sure the model started without issues as expected.

The output is on hera here: /scratch1/NCEPDEV/stmp2/Jessica.Meixner/FV3_RT/rt_66835
FYI: @LydiaStefanova-NOAA @barlage

@LydiaStefanova-NOAA
Copy link

I checked the soil temperature field from 20120101 for (rt_66835 vs original p7a) and compared with the difference in (correct land ICs vs originally used ICs)

The grids are different (regridded vs tiled), so cannot do a direct comparison, but a zoom in over a similar domain is below. It indicates that the change has gone through correctly.

a) fh000 rt_66835 test runs minus original p7a runs
image
b) tile 1 in /scratch1/NCEPDEV/stmp2/Michael.Barlage/spinup/mini7a/jarvis/2012010100/sfc_data.tile1.nc (correct land ICs)
minus
the ics used to initialise p7a /scratch1/NCEPDEV/nems/emc.nemspara/RT/NEMSfv3gfs/input-data-20210528/FV3_input_frac/BM7_IC/2012010100/gfs/C384_L127/INPUT/sfc_data.tile1.nc (land ICs used for original p7a runs)
image

@junwang-noaa junwang-noaa merged commit 68a27d4 into ufs-community:release/P7a Jun 8, 2021
@jiandewang
Copy link
Collaborator

thanks for everybody's quick action, I will start to re-run P7a now

@JessicaMeixner-NOAA JessicaMeixner-NOAA deleted the bug/p7ainput branch June 11, 2021 17:19
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.

4 participants