-
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] Update ufs-weather-model hash and UPP hash and use upp-addon-env spack-stack environment #1136
base: develop
Are you sure you want to change the base?
[develop] Update ufs-weather-model hash and UPP hash and use upp-addon-env spack-stack environment #1136
Conversation
…19) and UPP hash to 81b38a8 (Aug 13). Point to upp-addon-env spack-stack environment on Hera. Update srw_common.lua to use g2/3.5.1 and g2tmpl/1.13.0. Updated exregional_plot_allvars.py to handle updates made to postxconfig-NT-fv3lam.txt.
… for Hercules in build_hercules_intel.lua
… in build_jet_intel.lua
… in build_gaea_intel.lua
* .cicd/Jenkinsfile - Replaced cheyenne with derecho in commented sections and commented out Derecho. * doc/tables/Tests.csv - Removed nco-mode WE2E tests since these have been removed from the repository. * modulefiles/build_derecho_intel.lua - Update spack-stack environment to upp-addon-env.
… upp-addon-env in build_noaacloud_intel.lua
* .github/CODEOWNERS - Added Bruce Kropp as a reviewer from the Platform team. * modulefiles/build_orion_intel.lua - Udated spack-stack environment to upp-addon-env.
…ace SLP name and comment out REFC sections
* scripts/exregional_plot_allvars.py - Found method to successfully plot REFC using seek() and readline() pygrib commands * scripts/exregional_plot_allvars_diff.py - Same
…ther than the deprecated atmos_nthreads, to correct issue with threading in the weather model
While all 6 fundamental WE2E tests successfully pass following the latest updates (example given was run on Hercules):
the comprehensive tests are failing with the following error:
It isn't clear what the issue is. Since the indicated include file is part of the FV3, I'll reach out to them and see what might be happening for the tests that are now failing with this error message. |
@MichaelLueken Thanks for your work on the OMP problem. I came here to report a similar problem: I applied your changes and noticed there is still at least one fundamental test failing on Hera: |
@mkavulich That could certainly be the issue. Another potential issue is with respect to |
Issue #362 was opened in NOAA-GFDL/GFDL_atmos_cubed_sphere asking about the strange |
The OMP issue appears to be related to the configuration of the job_card in rocoto. In order for a WE2E test to properly run the For threading purposes, the number of nodes required is equal to the tasks per node (40 on Hera) divided by the number of threads used. This gives us a While submitting the |
…nsive WE2E tests to successfully run on Hera
… task to run on Orion
… build and run following update to Cray PE. Allow all comprehensive WE2E tests to successfully run on Gaea.
Derecho, Gaea, Hera, and Orion have been successfully updated and tested. There are still issues with Jet (high resolution tests needing 136 tasks are still in queue) and Hercules (AQM). Will continue to monitor these two issues tomorrow. |
…cst task to run on Hercules; updates for AQM WE2E
DESCRIPTION OF CHANGES:
g2/3.5.1
andg2tmpl/1.13.0
(these are required for UPP)postxconfig-NT-fv3lam.txt
file.Type of change
TESTS CONDUCTED:
DOCUMENTATION:
Updated documentation related to the table defining the WE2E tests currently available in the SRW App. The nco-mode WE2E tests had been removed, but were still present in the Tests.csv file. Removed these entries.
CHECKLIST