-
Notifications
You must be signed in to change notification settings - Fork 258
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
Make stochastic physics independent of fv3atm and its submodules #171
Make stochastic physics independent of fv3atm and its submodules #171
Conversation
…fv3atm for code review and testing
Regression testing against existing baseline on hera.intel and hera.gnu: one test fails ( rt_hera_intel.log Note: if the bugfix for NOAA-PSL/stochastic_physics#24 is applied to the current head of develop, the results are bit for bit identical to this PR for |
Creating new baselines on hera.gnu and hera.intel: all tests pass |
Manual testing on macosx.gnu and hera.intel: code compiles with |
Regression tests passed on orion.intel, hera.intel, hera.gnu; logs updated in the PR. |
….conf for cheyenne.intel from NCAR dtc/develop branch
Changes look good to me. |
The stochastic_physics repo has been updated. I'm cannot approve anything here, only leave comments. |
Are you sure @pjpegion? Does github let you do a review? If so, it should let you approve/comment/request changes. |
…pointer for stochastic_physics
Regression tests passed on wcoss_cray and wcoss_dell_p3; logs updated in the PR. |
I can only leave comments. |
Phil, I am not sure if you were added to ufs-weather-model developer group,
would you try again?
…On Thu, Aug 6, 2020 at 12:09 PM Phil Pegion ***@***.***> wrote:
I can only leave comments.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#171 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AI7D6TMVU6K7T5GXDM2TEOTR7LIS7ANCNFSM4PDVMJEA>
.
|
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.
Now I can approve.
Description
This PR and associated PRs make the stochastic_physics submodule independent of fv3atm and its submodules. For details, see NOAA-PSL/stochastic_physics#27.
Issue(s) addressed
This is required for updating/improving the cmake build system. I did not try to update the existing
CMakeLists.txt
, because I know that others are working on this. I only moved the existingCMakeLists.txt
fromFV3/stochastic_physics
tostochastic_physics
after making the code changes in NOAA-PSL/stochastic_physics#27.Testing
Regression testing was done on hera.intel in
REPRO
andPROD
mode for the two testsfv3_ccpp_stochy
andfv3_ccpp_ca
after applying the bugfix described in NOAA-PSL/stochastic_physics#24 to both the top of develop and the code in the PR. The results are bit-for-bit identical.Dependencies
NOAA-PSL/stochastic_physics#27
NOAA-EMC/fv3atm#145
#171