-
Notifications
You must be signed in to change notification settings - Fork 104
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
Adding a line to fixed the bug of scattering calculation #120
Comments
Your change is not correct. You should update the red part, you missed the middle one. Thanks very much for your help.
Change line " IF (IDECID == 17 ) THEN” as "IF (IDECID == 17 .or. IDECID == 20 .or. IDECID == 21) THEN”.
Thanks.
Kate
… On Apr 23, 2020, at 1:29 PM, WenMeng-NOAA ***@***.***> wrote:
Kate's change was submitted at the PR #112 <#112> at
#121 <#121>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#120 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/APJPDRBJAIH3I7E2LF2CW63ROCJKRANCNFSM4MPI3IIQ>.
|
Kate, You are right. The new change was added in the PR #122. |
The branch gefs_v12_release was committed at @0baf6bf with the bug fix of scattering calculation. |
guoqing-noaa
pushed a commit
to guoqing-noaa/UPP
that referenced
this issue
Apr 28, 2022
Added a new, optional external property, sparse. sparse points to a file (relative to external repository local directory) with git sparse checkout patterns (e.g., directory names to include). If the sparse keyword is included, enable sparse checkout for that repository and use git read-tree to perform the sparse checkout. Added a new test for sparse checkout. The test is a repo with two externals which are both tag2 of the simple_ext repo with one of them configured for a sparse checkout. The test makes sure that both externals have the correct files. Updated the documentation for the externals configuration file. User interface changes?: Yes New sparse keyword is optional with a blank default value which preserves current functionality. Fixes: NOAA-EMC#120 Testing: test removed: None unit tests: All passed system tests: All passed, including new test for sparse feature manual testing: Several manual tests of NASA test cases.
guoqing-noaa
pushed a commit
to guoqing-noaa/UPP
that referenced
this issue
Apr 28, 2022
EricJames-NOAA
pushed a commit
to EricJames-NOAA/UPP
that referenced
this issue
Dec 14, 2022
Change path to fix_am on Hera for build_regional
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Change line " IF (IDECID == 17 ) THEN" as
"IF (IDECID == 17 .or. IDECID == 20 .or. IDECID == 21) THEN"
The text was updated successfully, but these errors were encountered: