Skip to content
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

Merged
merged 3 commits into from
Oct 23, 2024
Merged

UFS-dev PR#226 #533

merged 3 commits into from
Oct 23, 2024

Conversation

grantfirl
Copy link
Collaborator

This is only a ccpp-physics submodule update for NCAR/ccpp-physics#1097

RT baseline changes are consistent with changes to ccpp physics changes.

@grantfirl
Copy link
Collaborator Author

RT results:

Output for arm_sgp_summer_1997_A_SCM_GFS_v17_p8_ugwpv1 is IDENTICAL to baseline
Output for arm_sgp_summer_1997_A_SCM_GFS_v16_RRTMGP is IDENTICAL to baseline
Output for arm_sgp_summer_1997_A_SCM_GFS_v16 is IDENTICAL to baseline
Output for arm_sgp_summer_1997_A_SCM_WoFS_v0 DIFFERS from baseline.
Output for arm_sgp_summer_1997_A_SCM_HRRR_gf DIFFERS from baseline.
Output for twpice_SCM_GFS_v17_p8_ugwpv1 is IDENTICAL to baseline
Output for twpice_SCM_GFS_v16_RRTMGP is IDENTICAL to baseline
Output for twpice_SCM_GFS_v16 is IDENTICAL to baseline
Output for twpice_SCM_WoFS_v0 DIFFERS from baseline.
Output for twpice_SCM_HRRR_gf DIFFERS from baseline.
Output for bomex_SCM_GFS_v17_p8_ugwpv1 is IDENTICAL to baseline
Output for bomex_SCM_GFS_v16_RRTMGP is IDENTICAL to baseline
Output for bomex_SCM_GFS_v16 is IDENTICAL to baseline
Output for bomex_SCM_WoFS_v0 is IDENTICAL to baseline
Output for bomex_SCM_HRRR_gf DIFFERS from baseline.
Output for astex_SCM_GFS_v17_p8_ugwpv1 is IDENTICAL to baseline
Output for astex_SCM_GFS_v16_RRTMGP is IDENTICAL to baseline
Output for astex_SCM_GFS_v16 is IDENTICAL to baseline
Output for astex_SCM_WoFS_v0 is IDENTICAL to baseline
Output for astex_SCM_HRRR_gf is IDENTICAL to baseline
Output for LASSO_2016051812_SCM_GFS_v17_p8_ugwpv1 is IDENTICAL to baseline
Output for LASSO_2016051812_SCM_GFS_v16_RRTMGP is IDENTICAL to baseline
Output for LASSO_2016051812_SCM_GFS_v16 is IDENTICAL to baseline
Output for LASSO_2016051812_SCM_WoFS_v0 is IDENTICAL to baseline
Output for LASSO_2016051812_SCM_HRRR_gf is IDENTICAL to baseline
Output for COMBLE_SCM_GFS_v17_p8_ugwpv1 is IDENTICAL to baseline
Output for COMBLE_SCM_GFS_v16_RRTMGP is IDENTICAL to baseline
Output for COMBLE_SCM_GFS_v16 is IDENTICAL to baseline
Output for COMBLE_SCM_WoFS_v0 DIFFERS from baseline.
Output for COMBLE_SCM_HRRR_gf DIFFERS from baseline.
Output for MOSAiC-AMPS_SCM_GFS_v17_p8_ugwpv1 is IDENTICAL to baseline
Output for MOSAiC-AMPS_SCM_GFS_v16_RRTMGP is IDENTICAL to baseline
Output for MOSAiC-AMPS_SCM_GFS_v16 is IDENTICAL to baseline
Output for MOSAiC-AMPS_SCM_WoFS_v0 DIFFERS from baseline.
Output for MOSAiC-AMPS_SCM_HRRR_gf is IDENTICAL to baseline
Output for gabls3_SCM_GFS_v16 is IDENTICAL to baseline
Output for arm_sgp_summer_1997_A_SCM_GFS_v17_p8 is IDENTICAL to baseline
Output for arm_sgp_summer_1997_A_SCM_HRRR DIFFERS from baseline.
Output for arm_sgp_summer_1997_A_SCM_RRFS_v1beta DIFFERS from baseline.
Output for arm_sgp_summer_1997_A_SCM_RAP DIFFERS from baseline.
Output for arm_sgp_summer_1997_A_SCM_GFS_v15p2 is IDENTICAL to baseline
Output for twpice_SCM_GFS_v17_p8 is IDENTICAL to baseline
Output for twpice_SCM_HRRR DIFFERS from baseline.
Output for twpice_SCM_RRFS_v1beta DIFFERS from baseline.
Output for twpice_SCM_RAP DIFFERS from baseline.
Output for twpice_SCM_GFS_v15p2 is IDENTICAL to baseline
Output for bomex_SCM_GFS_v17_p8 is IDENTICAL to baseline
Output for bomex_SCM_HRRR is IDENTICAL to baseline
Output for bomex_SCM_RRFS_v1beta is IDENTICAL to baseline
Output for bomex_SCM_RAP DIFFERS from baseline.
Output for bomex_SCM_GFS_v15p2 is IDENTICAL to baseline
Output for bomex_SCM_GFS_v16_debug is IDENTICAL to baseline
Output for astex_SCM_GFS_v17_p8 is IDENTICAL to baseline
Output for astex_SCM_HRRR is IDENTICAL to baseline
Output for astex_SCM_RRFS_v1beta is IDENTICAL to baseline
Output for astex_SCM_RAP is IDENTICAL to baseline
Output for astex_SCM_GFS_v15p2 is IDENTICAL to baseline
Output for LASSO_2016051812_SCM_GFS_v17_p8 is IDENTICAL to baseline
Output for LASSO_2016051812_SCM_HRRR is IDENTICAL to baseline
Output for LASSO_2016051812_SCM_RRFS_v1beta is IDENTICAL to baseline
Output for LASSO_2016051812_SCM_RAP is IDENTICAL to baseline
Output for LASSO_2016051812_SCM_GFS_v15p2 is IDENTICAL to baseline
ALL TESTS PASSED, BUT OUTPUT DIFFERS FROM BASELINE.

@dustinswales
Copy link
Collaborator

dustinswales commented Oct 21, 2024

@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?
The CI tests shouldn't fail when baselines disagree?

Update What is this? https://github.com/NCAR/ccpp-scm/actions/runs/11411899398/job/31756974040?pr=533#step:24:1664

@grantfirl
Copy link
Collaborator Author

@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? The CI tests shouldn't fail when baselines disagree?

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.

@mkavulich
Copy link
Collaborator

@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

@dustinswales
Copy link
Collaborator

@mkavulich Yeah, I didn't realize there was a new test being added in an earlier PR.
The artifact isn't being created because the RTs disagree with the baseline, but rather they are failing because the baseline does not exist.

@grantfirl
Copy link
Collaborator Author

@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.

@grantfirl grantfirl merged commit c1bdfb6 into NCAR:main Oct 23, 2024
21 of 24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants