-
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 nco version #1077
Conversation
|
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.
I just discovered that this change is required. Thanks for opening the PR, @RatkoVasic-NOAA!
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.
These changes look good to me and the fundamental tests successfully passed:
----------------------------------------------------------------------------------------------------
Experiment name | Status | Core hours used
----------------------------------------------------------------------------------------------------
grid_RRFS_CONUScompact_25km_ics_HRRR_lbcs_RAP_suite_RRFS_v1beta_2 COMPLETE 8.93
grid_RRFS_CONUS_25km_ics_FV3GFS_lbcs_FV3GFS_suite_GFS_v15p2_20240 COMPLETE 5.75
grid_RRFS_CONUS_25km_ics_FV3GFS_lbcs_FV3GFS_suite_GFS_v17_p8_plot COMPLETE 13.79
grid_RRFS_CONUScompact_25km_ics_HRRR_lbcs_HRRR_suite_HRRR_2024042 COMPLETE 34.47
grid_SUBCONUS_Ind_3km_ics_HRRR_lbcs_RAP_suite_WoFS_v0_20240423132 COMPLETE 22.59
grid_RRFS_CONUS_25km_ics_NAM_lbcs_NAM_suite_GFS_v16_2024042313282 COMPLETE 20.14
----------------------------------------------------------------------------------------------------
Total COMPLETE 105.67
Approving now.
The Jet pipeline is down, so I manually ran the coverage WE2E tests on that machine. All tests successfully passed:
|
Several tests failed on Orion due to bad nodes. Manual runs of the coverage WE2E tests on Orion have successfully passed:
Additionally, the automated tests on Derecho, Hera Intel, and Hercules have successfully passed. Once the last tests on Gaea and Hera GNU have passed, I will be able to merge this PR. |
The Gaea tests have successfully passed:
Merging this PR now. |
[develop] Update nco version (ufs-community#1077)
DESCRIPTION OF CHANGES:
Hera with Intel compiler was using system installed nco library (4.9.3 version). It was not noticed until sys admins removed read permissions to 4.9.3 version and installed new version (5.1.6).
Will use spack-stack installed nco (version 5.0.6), like all other machines/compilers.
Type of change
TESTS CONDUCTED:
ISSUE:
#1076
CHECKLIST
LABELS (optional):
A Code Manager needs to add the following labels to this PR: