-
Notifications
You must be signed in to change notification settings - Fork 165
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
Disable support for FAST-JX except for the Hg simulation #2379
Disable support for FAST-JX except for the Hg simulation #2379
Conversation
GeosCore/gc_environment_mod.F90 - Throw an error (right before the call to Init_FJX) if FAST-JX photolysis is activated for fullchem or carbon simulations. FAST-JX is currently only supported for Hg. Signed-off-by: Bob Yantosca <[email protected]>
Note that this update is only needed for use of GEOS-Chem in external models since GCHP and GC-Classic will both fail during the configure stage and thus the run-time error will never get triggered. This update prevents having to update build settings for all external models that use GEOS-Chem. |
GeosCore/gc_environment_mod.F90 - Now test if it is NOT a Hg simulation rather than testing if it is a fullchem or carbon simulation. Signed-off-by: Bob Yantosca <[email protected]>
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 good. Now approved!
All GEOS-Chem Classic integration tests passed: ==============================================================================
GEOS-Chem Classic: Execution Test Results
GCClassic #8c9d395 Updated CHANGELOG.md for clarity
GEOS-Chem #011885098 Change logic in FAST-JX error trap in gc_environment_mod.F90
HEMCO #2192e0e HEMCO 3.9.1 release
Using 24 OpenMP threads
Number of execution tests: 28
Submitted as SLURM job: 40086888
==============================================================================
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%%% All execution tests passed! %%%
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% All tests were zero-diff with the previous PR #2372, except:
|
All GCHP integration tests passed: ==============================================================================
GCHP: Execution Test Results
GCHP #8f8add6 Merge PR #426 (Throw error if FAST-JX is used w/ non-Hg mechanisms)
GEOS-Chem #011885098 Change logic in FAST-JX error trap in gc_environment_mod.F90
HEMCO #
Number of execution tests: 11
Submitted as SLURM job: 40087031
==============================================================================
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%%% All execution tests passed! %%%
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% All tests were zero-diff w/r/t the previous PR #2372. |
This merge brings PR #2379 (Disable support for FAST-JX except for the Hg simulation, by @yantosca) into the GEOS-Chem "no-diff-to-benchmark" simulation. This PR adds a runtime error trap that will halt execution if FAST-JX is selected for any simulation other than Hg. Eventually Hg will use Cloud-J. Signed-off-by: Bob Yantosca <[email protected]>
Name and Institution (Required)
Name: Bob Yantosca
Institution: Harvard + GCST
Describe the update
This is a corresponding PR to geoschem/GCClassic#62 and geoschem/GCHP#426, which are intended to disable support for FAST-JX except for the Hg simulation.
We have added an error trap in
GeosCore/gc_environment_mod.F90
, right before the call toInit_CMN_FJX
, which will stop the run if FAST-JX is enabled for simulations using thefullchem
,custom
, orcarbon
KPP mechanisms.Expected changes
This will be a no-diff-to-benchmark update. An error message will halt simulations (other than Hg) that use FAST-JX photolysis, with error output such as:
Related Github Issue