-
Notifications
You must be signed in to change notification settings - Fork 170
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 convection issues when meteorology is Grell Freitas scheme #2523
Fix convection issues when meteorology is Grell Freitas scheme #2523
Conversation
d130988
to
255a58d
Compare
I updated the PR description to include my contributions submitted via yuanjianz#1. |
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.
Hi @lizziel, thanks for the PR. I just caught a couple of typos in the warning messages in the rundir creation scripts. Otherwise looks good to go!
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.
Thanks @lizziel and @yuanjianz! Good to merge.
Great, I'll bring this in along with the GCHPctmEnv fixes PR. |
…eld definition change in Grell-Freitas convection
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
… this PR goes into 14.5.1
81c5b34
to
cee694d
Compare
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
This merge brings PR #2675 (Fixed the problem that GCHP cannot read meteorology field from yaml, by @yantosca) into the GEOS-Chem "no-diff-to-benchmark" development stream. According to @lizziel: I believe this fixes an issue brought in with the updates for GEOS-IT in dev/no-diff-to-benchmark (see PR #2523). It would only show up for GEOS-FP and GEOS-IT runs which we do not test in integration/AWS testing, and is not in any previously released versions. It therefore does not need an update to the changelog. Signed-off-by: Bob Yantosca <yantosca@seas.harvard.edu>
Name and Institution (Required)
Yuanjian Zhang (WashU), Lizzie Lundgren (Harvard)
Describe the update
See discussion #1409
GEOS-IT and GEOS-FP after Jun 2020 switch to Grell Freitas convection from RAS. Some meteorology field definitions are changed. This update accommodates these fields to geos-chem convection.
Reported by @viral211 first:
@yuanjianz:
Changes include:
geoschem_config.yml
Input_Opt
logical for whether to reconstruct convective precipitation flux. This switch is set automatically during run-time to true for GEOS-IT, true for GEOS-FP if start date is June 1 2020 or later, and false for all other cases.Input_Opt
logical to use an alternative method in convection which bypasses use of PFLCU and PFICU when necessary. This fixes the zero convective precipitation and high cloud base in runs using GEOS-FP (>=01Jun2020) or GEOS-IT.Related convection fixes are also applicable to simulations using met-fields generated with RAS. These are submitted via separate PR #2573. Those updates will impact the GEOS-Chem benchmark.
Expected changes
Lower cloud base and restored washout in geos-chem convection driven by GEOS-IT and GEOS-FP after Jun 2020. This will not affect the GEOS-Chem benchmark which uses MERRA2.
Related Github Issue
#2469