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

Make stochastic physics independent of fv3atm and its submodules #171

Conversation

climbfuji
Copy link
Collaborator

@climbfuji climbfuji commented Jul 21, 2020

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 existing CMakeLists.txt from FV3/stochastic_physics to stochastic_physics after making the code changes in NOAA-PSL/stochastic_physics#27.

Testing

Regression testing was done on hera.intel in REPRO and PROD mode for the two tests fv3_ccpp_stochy and fv3_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

@climbfuji
Copy link
Collaborator Author

Regression testing against existing baseline on hera.intel and hera.gnu: one test fails (fv3_ccpp_ca) as expected because of the bugfix for NOAA-PSL/stochastic_physics#24 for hera.intel; for hera.gnu, this test is not run and all tests pas.

rt_hera_intel.log
rt_hera_gnu.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 fv3_ccpp_ca.

@climbfuji
Copy link
Collaborator Author

Creating new baselines on hera.gnu and hera.intel: all tests pass

rt_hera_intel_create.log
rt_hera_gnu_create.log

@climbfuji
Copy link
Collaborator Author

Manual testing on macosx.gnu and hera.intel: code compiles with compile.sh as well as with compile_cmake.sh. Code runs and gives same results for fv3_ccpp_stochy compared to head of develop on macsox.gnu.

@climbfuji
Copy link
Collaborator Author

Regression tests passed on orion.intel, hera.intel, hera.gnu; logs updated in the PR.

….conf for cheyenne.intel from NCAR dtc/develop branch
@pjpegion
Copy link
Collaborator

pjpegion commented Aug 6, 2020

Changes look good to me.

@pjpegion
Copy link
Collaborator

pjpegion commented Aug 6, 2020

The stochastic_physics repo has been updated. I'm cannot approve anything here, only leave comments.

@aerorahul
Copy link
Contributor

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.

@climbfuji
Copy link
Collaborator Author

Regression tests passed on wcoss_cray and wcoss_dell_p3; logs updated in the PR.

@pjpegion
Copy link
Collaborator

pjpegion commented Aug 6, 2020

I can only leave comments.

@junwang-noaa
Copy link
Collaborator

junwang-noaa commented Aug 6, 2020 via email

Copy link
Collaborator

@pjpegion pjpegion left a 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.

@junwang-noaa junwang-noaa merged commit 92564f8 into ufs-community:develop Aug 6, 2020
pjpegion pushed a commit to NOAA-PSL/ufs-weather-model.p7b that referenced this pull request Jul 20, 2021
epic-cicd-jenkins pushed a commit that referenced this pull request Apr 17, 2023
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.

5 participants