-
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 a WE2E test for AQM #840
[develop] Add a WE2E test for AQM #840
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.
@chan-hoo I was able to successfully run the new WE2E test for AQM on Hera:
----------------------------------------------------------------------------------------------------
Experiment name | Status | Core hours used
----------------------------------------------------------------------------------------------------
aqm_grid_AQM_NA13km_suite_GFS_v16 COMPLETE 981.50
----------------------------------------------------------------------------------------------------
Total COMPLETE 981.50
Approving this PR now.
@RatkoVasic-NOAA, @JeffBeck-NOAA, @MichaelLueken, thank you for your approvals !!! |
@chan-hoo The automated Jenkins tests passed on Cheyenne, Hera, and Jet. The coverage WE2E tests were manually ran on Orion and all successfully passed:
Now moving forward with merging this work. |
I didn't see this before it got merged but I wanted to say this is a great addition! Thanks @chan-hoo! |
DESCRIPTION OF CHANGES:
Type of change
TESTS CONDUCTED:
./devbuild.sh -p=hera -a=ATMAQ
aqm_grid_AQM_NA13km_suite_GFS_v16
ISSUE:
Fixes issue mentioned in #815
CHECKLIST
CONTRIBUTORS:
@natalie-perlin