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

GEOS-Chem Transport Tracers simulation GEOS #2058

Conversation

lizziel
Copy link
Contributor

@lizziel lizziel commented Dec 5, 2023

Name and Institution (Required)

Name: Lizzie Lundgren
Institution: Harvard University

Describe the update

This PR enables usage of the GEOS-Chem transport tracer simulation within GEOS. Config files are provided in new directory run/GEOS/TransportTracers. Important notes:

  1. All standard transport tracers diagnostics are included in the HISTORY.rc file except the meteorology level edge collection. This collection contains diagnostics on 73 levels which is currently not functional when run in GEOS due to a GEOS-only code block that flips GEOS-Chem history exports in MAPL to be TOA level = 1.
  2. Currently it must be run with USE_EXTDATA2G set to .FALSE. in CAP.rc.
  3. A draft version of GEOSCHEMchem_ExtData.yaml is included in this PR but is not yet functional.

Expected changes

No impact on full chemistry or offline model runs.

Reference(s)

None

Related Github Issue(s)

None

lizziel and others added 10 commits July 21, 2023 13:31
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
With this commit the GEOS model builds and runs using GEOS-Chem transport
tracer simulation with new config files stored in
geos-chem/run/GEOS/TransportTracers. The following is currently required
to run successfully:
1. CEDSv2 must be turned off in HEMCO_Config.rc
2. ExtData2G must be turned off in CAP.rc

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Specific humidity will be available in transport tracer sim checkpoints
for purposes of post-processing unit conversions.

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
This avoids an out-of-bound issue when vertically flipping
diagnostics with 73 levels in GEOS.

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
@lizziel lizziel marked this pull request as ready for review December 12, 2023 21:57
@lizziel lizziel requested a review from msulprizio December 12, 2023 21:57
@lizziel lizziel added no-diff-to-benchmark This update will not change the results of fullchem benchmark simulations category: Interface to External Model Related to GEOS-Chem updates needed to interface with other models specialty simulations labels Dec 12, 2023
@lizziel lizziel added this to the 14.3.0 milestone Dec 12, 2023
@lizziel lizziel changed the base branch from main to dev/14.3.0 December 12, 2023 21:58
Copy link
Contributor

@msulprizio msulprizio left a comment

Choose a reason for hiding this comment

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

Good to merge. These updates will not impact the GCClassic or GCHP full-chemistry benchmark simulations.

@lizziel lizziel changed the base branch from dev/14.3.0 to dev/no-diff-to-benchmark December 14, 2023 21:21
@lizziel lizziel removed this from the 14.3.0 milestone Dec 14, 2023
@yantosca
Copy link
Contributor

I've merged this up locally to dev/no-diff-to-benchmark and am running the integration tests. Thanks @lizziel and @msulprizio!

@yantosca
Copy link
Contributor

All GEOS-Chem Classic integration tests passed:

==============================================================================
GEOS-Chem Classic: Execution Test Results

GCClassic #8774cc3 GEOS-Chem submod update: Reduce timers to essential list only
GEOS-Chem #c5dca3a84 Merge PR #2058 (GEOS-Chem Transport Tracers simulation GEOS)
HEMCO     #a5d5169 HEMCO 3.7.2 release

Using 24 OpenMP threads
Number of execution tests: 27

Submitted as SLURM job: 12681154
==============================================================================

Summary of test results:
------------------------------------------------------------------------------
Execution tests passed: 25
Execution tests failed: 2
Execution tests not yet completed: 0

All integration tests were zero-diff w/r/t PR #2069 (new Hg restart files) except for:

  • TOMAS (failed)
  • APM (parallelization error)
  • RRTMG (numerical noise in diagnostic outputs)

@yantosca
Copy link
Contributor

All GCHP integration tests passed:

==============================================================================
GCHP: Execution Test Results

GCHP      #7b7bfa3 PR #334 post-merge update: Edit CHANGELOG.md
GEOS-Chem #c5dca3a84 Merge PR #2058 (GEOS-Chem Transport Tracers simulation GEOS)
HEMCO     #a5d5169 HEMCO 3.7.2 release

Number of execution tests: 5

Submitted as SLURM job: 12681186
==============================================================================
 
Execution tests:
------------------------------------------------------------------------------


%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%%%  All execution tests passed!  %%%
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%

All GCHP integration tests were zero-diff w/r/t PR #2069.

@yantosca yantosca merged commit c5dca3a into geoschem:dev/no-diff-to-benchmark Dec 15, 2023
lizziel added a commit that referenced this pull request Jan 5, 2024
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
@yantosca yantosca added this to the 14.3.0 milestone Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Interface to External Model Related to GEOS-Chem updates needed to interface with other models no-diff-to-benchmark This update will not change the results of fullchem benchmark simulations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants