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

Fix RAP (Grell Freitas) decomp b4b issues #156

Merged

Conversation

SamuelTrahanNOAA
Copy link
Collaborator

Fixes a bug in the GF scheme that prevented bit-for-bit reproducibility. This is the same as:

NCAR#942

grantfirl and others added 30 commits December 7, 2021 10:39
…lux_bug

Fix scm specified surface flux bug
@@ -2639,7 +2641,7 @@ subroutine cup_dd_edt(ierr,us,vs,z,ktop,kbcon,edt,p,pwav, &
prop_c=0. !10.386
alpha3 = 0.75
beta3 = -0.15
pefc=0.
pefc(:)=0.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coding question. Is it important that pefc has an i component since it is output from this subroutine? I am trying to understand why pefc needs to have its component defined, but pef and pefb do not.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In cu_gf_deep_run, ccnloss needs to have a different value for each i. There are several ways to do that, and this is the one Dom chose.

@SamuelTrahanNOAA SamuelTrahanNOAA merged commit 340b177 into NOAA-GSL:gsl/develop Jun 14, 2022
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.

8 participants