-
Notifications
You must be signed in to change notification settings - Fork 1
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
Added RRTMG and RATS toggles and settings #649
base: develop
Are you sure you want to change the base?
Conversation
AGCM.rc.tmpl
Outdated
@@ -865,6 +865,8 @@ DIURNAL_BIOMASS_BURNING: yes | |||
RATS_PROVIDER: @RATS_PROVIDER # options: PCHEM, GMICHEM, STRATCHEM (Radiatively active tracers) | |||
AERO_PROVIDER: @AERO_PROVIDER # options: GOCART2G, MAM, none (Radiatively active aerosols) | |||
ANALYSIS_OX_PROVIDER: @OANA_PROVIDER # options: PCHEM, GMICHEM, STRATCHEM, GOCART | |||
CH4_PROVIDER: @CH4_PROVIDER # options: GOCART, GHG, none (GHG = RRG) |
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.
Looks like no setup script is providing @CH4_PROVIDER
(and I'm guessing @CO2_PROVIDER
), so you probably need edits in those.
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.
Ah. Yes. Thanks. Will do.
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.
@mathomp4 OK. Done. Do I need to create a new PR? The changes are pushed to the mslong1/rrg+rad branch
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.
No. You did right, though I found a small bug. I just pushed a fix and we'll see how CI goes.
@msl3v can you please provide a description on what is intended behind this PR and what configurations these changes will support? Will there be corresponding changes to the setup scripts? |
There are two primary changes in this set of PRs.
The setup will be (should have been already :-/) modified to populate relevant AGCM.rc settings to default values that preserve a zero-diff result. |
@msl3v Well, first up the GNU build is not happy:
here is a permalink for this: https://github.com/GEOS-ESM/RRG/blob/2324ec4cdb54cc961f0e868d762ad319e6448da7/RRGMod.F90#L350 I honestly do not know why it's unhappy. You are just declaring a pointer. @tclune any ideas? But beyond that, Intel builds but the run is dying with:
and the issue seems to be that |
I don't know the precise problem, but do have some general recommendations. (And some might have an impact.)
|
I'm going to try building the RRG model with GNU on discover. See if I can reproduce this... |
MSL