-
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] Add comprehensive.gaea-c5 test suite. #944
[develop] Add comprehensive.gaea-c5 test suite. #944
Conversation
…rom_NOMADS_ics_FV3GFS_lbcs_FV3GFS test to pass and add comprehensive.noaacloud test suite.
Comprehensive testing has completed on Orion:
Rerunning the one failed test,
|
…ehensive.gaea-c5.
Comprehensive testing on Gaea C5 has completed:
Opening PR for review now. |
Rerunning the
|
Rerunning the
Tests have successfully passed for Gaea C5, Hera GNU, Hercules, and Orion. Awaiting Jet results now. |
The tests have successfully passed on Jet:
Merging now. |
DESCRIPTION OF CHANGES:
comprehensive.gaea-c5
, which is a symlink tocomprehensive.orion
.parm/wflow/verify_pre.yaml
file has been updated to give more time to APCP and reset the amount of time given for ASNOW.comprehensive.noaacloud
).load("wget")
was added to the build_orion_intel.lua modulefile to allow theget_from_NOMADS_ics_FV3GFS_lbcs_FV3GFS
to successfully retrieve data from NOMADS.valid_param_vals.yaml
has been updated to include the"reuse"
option invalid_vals_PREEXISTING_DIR_METHOD
. Documentation is still present in theush/config_defaults.yaml
file and if users choose to use this option, then the test will fail since it isn't in thevalid_param_vals.yaml
file.Type of change
TESTS CONDUCTED:
DOCUMENTATION:
None
ISSUE:
Fixes #943
CHECKLIST