-
Notifications
You must be signed in to change notification settings - Fork 119
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
[develop] Enable UPP 2d decomposition #917
[develop] Enable UPP 2d decomposition #917
Conversation
…2d decomposition.
…CRTM coefficients and add CRTM_DIR to Hera machine yaml file.
…st. Add itasks: 2 to the model_configure file for the grid_RRFS_CONUS_25km_ics_FV3GFS_lbcs_FV3GFS_suite_GFS_v15p2 test for inline post 2d decomposition and add numx: 2 to the itag namelist option file for the grid_RRFS_CONUScompact_25km_ics_HRRR_lbcs_RAP_suite_RRFS_v1beta test for offline post 2d decomposition.
…efficient directory.
…fficient directory and cleaned up comment in build_derecho_intel.lua.
…icient directory.
…e_file unit tests to pass.
…efficient directory.
…fficient directory.
… up-to-date versions. Use configxpost-NT-fv3lam.txt from ufs-weather-model/tests/parm instead of configxpost-NT-fv3lam_rrfs.txt until CRTM issue is addressed on Derecho. Add Derecho label to Jenkinsfile and reactivate comprehensive testing in Jenkins.
b8a6b57
to
e720541
Compare
…Workflow.rst with Gillian's suggestions. Co-authored-by: Gillian Petro <[email protected]>
…Workflow.rst with Gillian's suggestions. Co-authored-by: Gillian Petro <[email protected]>
…unity/config.grid_RRFS_CONUS_25km_ics_FV3GFS_lbcs_FV3GFS_suite_GFS_v15p2.yaml with Gillian's suggestions. Co-authored-by: Gillian Petro <[email protected]>
…unity/config.grid_RRFS_CONUScompact_25km_ics_HRRR_lbcs_RAP_suite_RRFS_v1beta.yaml with Gillian's suggestions. Co-authored-by: Gillian Petro <[email protected]>
…Workflow.rst with Gillian's suggestions. Co-authored-by: Gillian Petro <[email protected]>
…Workflow.rst with Gillian's suggestions. Co-authored-by: Gillian Petro <[email protected]>
Co-authored-by: Gillian Petro <[email protected]>
…ition documentation in ush/config_defaults.yaml and added prints regarding resetting NUMX and ITASKS in tests/WE2E/run_WE2E_tests.py.
@gspetro-NOAA - Thank you very much for reviewing these changes! I have committed the suggestions that you have made to the documentation, replaced the lowercase d with uppercase D for ITASKS in |
… reset in run_WE2E_tests.py and replace lowercase d with uppercase D in grid_RRFS_CONUScompact_25km_ics_HRRR_lbcs_RAP_suite_RRFS_v1beta documentation.
It looks like we need to add the line:
to the noaacloud machine file for the fundamental tests to run on the cloud. @EdwardSnyder-NOAA said that we did this for the release in PR #937 and should do that in
|
…ml to allow for workflow generation on NOAA Cloud platforms.
Thank you very much for running the fundamental tests on NOAA Cloud platforms, @gspetro-NOAA! I have added the |
There was a problem hiding this 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! Approving...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one question before I approve
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry I let this slide off my plate: I'll approve now so I don't hold up this PR any longer: my only concern was a bit of confusion on two different parameters controlling the same thing (one for inline post, one for offline post). But this is a documentation issue if it is an issue at all.
…used for offline post 2D decomposition
On Jet, the
This also shows that the code in this branch is ready to run on Rocky8, since the xjet partition is required to run the forecast job. One caveat is that the Jenkins pipeline for Jet will need to be set to one of the login nodes that has been transitioned to Rocky8, or else the Once a successful run of the |
Manually submitting the Jenkins coverage WE2E tests on Derecho successfully passed:
Moving forward with merging this work now. |
DESCRIPTION OF CHANGES:
With recent changes made to the UFS-WM (PR #1823), it is now possible to see in the log files that UPP 2d decomposition is being performed for inline post. To this end, the UFS-WM hash has been updated to 020e783 (October 27, 2023), the UPP hash was updated to fae617b (October 6, 2023), and the UFS_UTILS hash was updated to dc0e4a6 (November 6, 2023).
With the updated UPP hash, the old
postxconfig-NT-fv3lam.txt
file had been removed. Wen suggested using thepostxconfig-NT-fv3lam_rrfs.txt
file instead. Unfortunately, due to issues with the CRTM on Derecho, we are unable to move forward with this postxconfig file (this file contains assimilated radiances). The ufs-weather-model/tests/parm directory contains a copy of the oldpostxconfig-NT-fv3lam.txt
file. To expedite moving forward with this update, I'm currently pointing to this copy of the postxconfig file. Once the issue with the CRTM on Derecho has been corrected, the SRW App repository can transition to thepostxconfig-NT-fv3lam_rrfs.txt
file.Changes to enable 2d decomposition include:
itasks
to themodel_configure
file (values greater than 1 enable 2d decomposition in inline post).numx
to the end of the&NAMPGB
namelist options (values of numx greater than 1 enable 2d decomposition in offline post).itasks
to the list of variables to be added to themodel_configure
file.ITASKS: 2
to enable inline post 2d decomposition.NUMX: 2
to enable offline post 2d decomposition.The ability to run comprehensive tests has been added back into the Jenkinsfile, as well as the ability to run the automated Jenkins tests on Derecho.
Type of change
TESTS CONDUCTED:
The 2D decomposition does not work with GNU compilers (the
run_post
andrun_fcst
tasks fail on Hera GNU with double free or corruption errors). Disabling 2D decomposition for GNU compilers will allow the updated 2D decomposition WE2E tests to run on Hera GNU.DEPENDENCIES:
None
DOCUMENTATION:
Documentation has been updated in the WE2E test configuration files, the
ConfigWorkflow.rst
chapter, andconfig_defaults.yaml
.ISSUE:
Resolves #480
CHECKLIST