Skip to content

[develop] Sync ufs-weather-model hash to 826f6ff (April 6) version - #1371

Merged
MichaelLueken merged 5 commits into
ufs-community:developfrom
MichaelLueken:feature/hash_update_Apr_2026
May 5, 2026
Merged

[develop] Sync ufs-weather-model hash to 826f6ff (April 6) version#1371
MichaelLueken merged 5 commits into
ufs-community:developfrom
MichaelLueken:feature/hash_update_Apr_2026

Conversation

@MichaelLueken

@MichaelLueken MichaelLueken commented Apr 10, 2026

Copy link
Copy Markdown
Collaborator

DESCRIPTION OF CHANGES:

  • Externals.cfg - Sync ufs-weather-model hash to 826f6ff (April 6)
  • parm/field_table.FV3_RRFS_v1beta - Updated to allow grid_RRFS_CONUS_25km_ics_FV3GFS_lbcs_FV3GFS_suite_RRFS_v1beta WE2E test to run

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

TESTS CONDUCTED:

  • derecho.intel - AQM, Coverage (+ RRFS_v1beta), UFS Fire, and Fall Ozone use case tests
  • gaeac6.intel - AQM, Comprehensive, and UFS Fire WE2E tests
  • hercules.intel - AQM, Comprehensive, UFS Fire, and AEROMMA use case tests
  • orion.intel - AQM, Comprehensive, and UFS Fire WE2E tests
  • ursa.gnu - Coverage and UFS Fire WE2E tests
  • ursa.intel - AQM, Comprehensive, and UFS Fire WE2E tests

DEPENDENCIES:

N/A

DOCUMENTATION:

N/A

ISSUE:

N/A

CHECKLIST

  • My code follows the style guidelines in the Contributor's Guide
  • I have performed a self-review of my own code using the Code Reviewer's Guide
  • My changes do not require updates to the documentation (explain).
    • This PR is only updating the weather model hash
  • My changes generate no new warnings
  • New and existing tests pass with my changes

 * Externals.cfg - Sync ufs-weather-model hash to 826f6ff (April 6)
 * parm/field_table.FV3_RRFS_v1beta - Updated to allow grid_RRFS_CONUS_25km_ics_FV3GFS_lbcs_FV3GFS_suite_RRFS_v1beta WE2E test to run
@MichaelLueken

Copy link
Copy Markdown
Collaborator Author

All testing has been completed. Will open this PR now., but PR #1372 has priority over this PR.

Comment thread parm/field_table.FV3_RRFS_v1beta
Comment thread parm/field_table.FV3_RRFS_v1beta Outdated
…ta to remove misspellings and additional blank spaces while copying from the ufs-weather-model.

@RatkoVasic-NOAA RatkoVasic-NOAA left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

MichaelLueken and others added 3 commits April 30, 2026 16:48
* doc/UsersGuide/BackgroundInfo/Components.rst - Update path to Rocoto documentation
* doc/UsersGuide/CustomizingTheWorkflow/DefineWorkflow.rst - Same
* doc/UsersGuide/Reference/Glossary.rst - Same
@MichaelLueken MichaelLueken added the run_we2e_coverage_tests Run the coverage set of SRW end-to-end tests label May 4, 2026
@MichaelLueken

Copy link
Copy Markdown
Collaborator Author

The Jenkins tests have successfully passed on Azure, Gaea C6, Orion, AWS PW, Ursa GNU, and Ursa Intel.

The tests are still running on Derecho, but given that tasks are sitting in queue for long periods of time, the coverage tests will likely need to be manually run on that platform.

On Hercules, the Functional WorkflowTaskTests stage failed. The message in the err_wrap.*.err file is:

/work2/noaa/epic/role-epic/jenkins/workspace/s-srweather-app_pipeline_PR-1371/hercules/.cicd/scripts/srw_ftest.sh: line 96: uw: command not found

In the .cicd/scripts/srw_ftest.sh file, the conda environment is deactivated, then the srw_app conda environment is activated:

conda deactivate
conda activate srw_app

So it isn't clear what the issue is. I will try running the srw_ftest.sh script on /work2 on Hercules and see if I can replicate the failure.

@MichaelLueken

Copy link
Copy Markdown
Collaborator Author

I'm unable to reproduce the issue on Hercules. My running of the Functional WorkflowTaskTests on Hercules successfully passed:

# Try hercules with the first few simple SRW tasks ...
run_make_grid: COMPLETE
run_get_ics: COMPLETE
run_get_lbcs: COMPLETE
run_make_orog: COMPLETE
run_make_sfc_climo: COMPLETE
run_make_ics: COMPLETE
run_make_lbcs: COMPLETE
run_fcst: COMPLETE
run_post: COMPLETE

The coverage WE2E tests also successfully ran:

----------------------------------------------------------------------------------------------------
Experiment name                                                  | Status    | Core hours used 
----------------------------------------------------------------------------------------------------
grid_RRFS_CONUS_25km_ics_FV3GFS_lbcs_FV3GFS_suite_GFS_v17_p8_plot  COMPLETE              13.58
grid_RRFS_CONUS_3km_ics_FV3GFS_lbcs_FV3GFS_suite_GFS_v15p2_202605  COMPLETE             503.79
grid_RRFS_CONUS_25km_ics_FV3GFS_lbcs_FV3GFS_suite_HRRR_2026050412  COMPLETE              16.83
grid_RRFS_NA_13km_ics_FV3GFS_lbcs_FV3GFS_suite_RAP_20260504125718  COMPLETE             126.25
grid_SUBCONUS_Ind_3km_ics_NAM_lbcs_NAM_suite_GFS_v16_202605041257  COMPLETE              43.90
specify_EXTRN_MDL_SYSBASEDIR_ICS_LBCS_20260504125759               COMPLETE               7.29
2019_hurricane_lorenzo_20260504125823                              COMPLETE              75.74
2020_denver_radiation_inversion_20260504125845                     COMPLETE              74.40
grid_SUBCONUS_Ind_3km_ics_FV3GFS_lbcs_FV3GFS_suite_WoFS_v0_202605  COMPLETE              48.10
----------------------------------------------------------------------------------------------------
Total                                                              COMPLETE             909.88
COL_NAME:                           FCST_MODEL                           REF_MODEL N_INIT N_TERM N_VLD SS_INDEX
 GO_INDEX FV3_WoFS_v0_SUBCONUS_3km_test_mem000 FV3_WoFS_v0_SUBCONUS_Ind_3km_mem000      1     15    15  0.99617

Will attempt to run the WE2E coverage tests via Jenkins, but I am unable to reproduce the issue and there have been no changes to Hercules, so I am unable to debug this issue.

@MichaelLueken

Copy link
Copy Markdown
Collaborator Author

The coverage WE2E tests have successfully passed on Derecho.

Attempting to run the coverage tests on Hercules via Jenkins. Once complete, I will merge this PR, whether the tests pass or fail (given that the issue is with the EPIC role account, which I don't have access to on Hercules or Orion).

@MichaelLueken

Copy link
Copy Markdown
Collaborator Author

I think I see the issue on Hercules. While building the conda environments, I'm seeing the following:

Preparing transaction: ...working... done
Verifying transaction: ...working... done
Executing transaction: ...working... done
Rolling back transaction: ...working... done

[Errno 122] Disk quota exceeded
()

@MichaelLueken

Copy link
Copy Markdown
Collaborator Author

Given that it is currently impossible to run the SRW App on Hercules under the EPIC role account, but the manual run on the WE2E coverage tests successfully passed, I will now move forward with merging this PR.

@MichaelLueken
MichaelLueken merged commit 77602ed into ufs-community:develop May 5, 2026
7 checks passed
@github-project-automation github-project-automation Bot moved this from In Review to Done in SRW App's PR Reviewers May 5, 2026
@MichaelLueken
MichaelLueken deleted the feature/hash_update_Apr_2026 branch May 5, 2026 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run_we2e_coverage_tests Run the coverage set of SRW end-to-end tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants