Wrapper for ccpp-physics #812 #972
Conversation
|
FYI, the following RTs failed on hera.intel. I'm not sure if this represents only those that include the SAMF deep or shallow and/or SATMEDMF schemes or not since I'm not as familiar with the tests as others. cpld_control_p7 001 failed in check_result |
|
It looks like the only test that failed the run_test only was control_c384gdas_wav which had the following error during run:
|
|
Looks like a disk quota issue. We got an email this morning, once again, that stmp2 is over quota.
… On Dec 22, 2021, at 9:41 AM, Grant Firl ***@***.***> wrote:
It looks like the only test that failed the run_test only was control_c384gdas_wav which had the following error during run:
forrtl: severe (38): error during write, unit 50, file /scratch1/BMC/gmtb/Grant.Firl/FV3_RT/stmp2/Grant.Firl/FV3_RT/rt_15495/control_c384gdas_wav/PET473.ESMF_LogFile
—
Reply to this email directly, view it on GitHub <#972 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AB5C2RNY36MO7E5KHTL72OTUSH5R3ANCNFSM5KRSVZJQ>.
Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you are subscribed to this thread.
|
OK, thanks. I'll clear some space and try that test again to make sure. |
|
Confirmed that the fail run test was a disk quota issue. Add control_c384gdas_wav to the list of failed check_results. |
|
The control_flake froze for a half hour without even starting to execute. I didn't even get the initial NEMS message. On a second try, it finished in four minutes. I'm going to assume that was due to a hardware or OS issue unless it happens again. |
|
Two more tests froze. I attached to a running process stuck in this state, and it appears the PMPI_Init_thread is freezing. That's part of the MPI initialization. This failure is definitely not caused by our code. |
|
Code was committed in PR#981. |
ALL CREDIT TO @rmontuoro FOR THIS PR. This is only a wrapper for testing/merging purposes. All text below is from #964
PR Checklist
Ths PR is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR. Please consult the ufs-weather-model wiki if you are unsure how to do this.
This PR has been tested using a branch which is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR
An Issue describing the work contained in this PR has been created either in the subcomponent(s) or in the ufs-weather-model. The Issue should be created in the repository that is most relevant to the changes in contained in the PR. The Issue and the dependent sub-component PR
are specified below.
Results for one or more of the regression tests change and the reasons for the changes are understood and explained below.
New or updated input data is required by this PR. If checked, please work with the code managers to update input data sets on all platforms.
Description
This PR implements a vertically-integrated formula to properly compute tracer column mass and ensure mass conservation for all tracers except TKE. See issue NCAR/ccpp-physics#811 for details.
This PR will change the baseline for all regression tests using Scale-Aware Mass-Flux (SAMF) shallow or deep convective schemes, as well as the Scale-aware TKE-based Moist Eddy-Diffusivity Mass-Flux (SATMEDMF) PBL scheme.
Issue(s) addressed
Testing
Dependencies