-
Notifications
You must be signed in to change notification settings - Fork 259
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
re-enable c48-5deg warmstart test #1822
Comments
@NickSzapiro-NOAA Are you able to work on this now? I want to include the required files in the input-data update I will be making for the Bu angle fix. We need new ICs (really warmstart) files at These are generated by running the |
@DeniseWorthen restarts from cpld_control_c48 are available on hera at /scratch1/NCEPDEV/nems/Nick.Szapiro/tasks/warmStart_c48-5deg/IC |
@NickSzapiro-NOAA Thanks. I've repopulated the input-data directory here Now can you re-enable the two c48 cpld tests (warmstart and restart) and create a warm start baseline after pointing to this location for the input and verify both the warm and restart tests pass? |
cpld*c48 tests now pass on hera. See /scratch1/NCEPDEV/nems/Nick.Szapiro/tasks/warmStart_c48-5deg/ufs-weather-model/tests |
@NickSzapiro-NOAA are you going to create a UFS WM PR to include the tests in RT? |
@junwang-noaa I think @DeniseWorthen already did in #2235 |
Yes, because re-enabling the test requires new input data, I added it to the PR which will fix input grid data for CICE. |
Description
The C48-5deg warmstart test was disabled in PR #1812. It needs to be re-enabled.
Solution
Once PR #1777 and #1807 are merged, a new set of warm start ICs should be created and added to the input-data directory. This will allow the test to be re-enabled.
Alternatives
Related to
The text was updated successfully, but these errors were encountered: