-
Notifications
You must be signed in to change notification settings - Fork 249
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
Clean up of stochastic physics #823
Labels
enhancement
New feature or request
Comments
14 tasks
@pjpegion can we close the issue? |
Yes!
…On Tue, Oct 5, 2021 at 2:52 PM Jun Wang ***@***.***> wrote:
@pjpegion <https://github.com/pjpegion> can we close the issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#823 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AJIRVJEMSZMLV3S6H2OA2ZLUFNQQDANCNFSM5ETZZAIQ>
.
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>.
--
Phil Pegion (he/him/his)
Physical Scientist
NOAA/Physical Sciences Laboratory
(303) 497-7897
***@***.***
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
The current stochastic physics code was extracted from the legacy GFS spectral model, and contains many unused subroutines variables. In addition the cellular automata portion of the code does not produce the same answer using different processor layout.
Solution
Removal of unused code, consolidation of many individual subroutine files into a few modules, and switch from f77 to f90. To address the CA issue, there is a new random number generator which is as fast as mersenne twister, but has less overhead in the seed calculation.
Alternatives
none
Dependencies
requires an update to ccpp-physics, fv3atm, and stochastic_physics
The text was updated successfully, but these errors were encountered: