-
Notifications
You must be signed in to change notification settings - Fork 49
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
UFS-dev PR#226 #533
UFS-dev PR#226 #533
Conversation
RT results: Output for arm_sgp_summer_1997_A_SCM_GFS_v17_p8_ugwpv1 is IDENTICAL to baseline |
@grantfirl @scrasmussen @mkavulich It seems that the CI tests are now failing when the baseline comparison fails, and consequentially not creating new github artifacts that need to be used to get the tests passing again? Update What is this? https://github.com/NCAR/ccpp-scm/actions/runs/11411899398/job/31756974040?pr=533#step:24:1664 |
I added a new test that runs schemes in the GFS_debug file for Dom. Let's concentrate on PRs in order and debug this when we get to it. |
@dustinswales I don't think that's what's keeping the artifacts from being created, I think it's the new failing "GFS_debug" test. You can see that the single-precision artifacts were created. I do see the same message you do ('No artifacts will be uploaded.'). I honestly don't know what that's about, because they clearly are being uploaded. You can see the same message for all tests in the last PR here: https://github.com/NCAR/ccpp-scm/actions/runs/11446706947 |
@mkavulich Yeah, I didn't realize there was a new test being added in an earlier PR. |
@dustinswales @scrasmussen @mkavulich This is ready to review. Several RTs that use GF convection and MYNN PBL change results. Therefore, we'll need to upload the artifacts from this. |
This is only a ccpp-physics submodule update for NCAR/ccpp-physics#1097
RT baseline changes are consistent with changes to ccpp physics changes.