Skip to content

Enable use of ExtData2G in GCHP Transport Tracers simulation#41

Open
lizziel wants to merge 7 commits intogchp/devfrom
bugfix/vector_exports_in_extdata2G
Open

Enable use of ExtData2G in GCHP Transport Tracers simulation#41
lizziel wants to merge 7 commits intogchp/devfrom
bugfix/vector_exports_in_extdata2G

Conversation

@lizziel
Copy link

@lizziel lizziel commented Oct 29, 2025

This PR implements a fix from Ben Auer (NASA GMAO) for a bug found by me and reported in GEOS-ESM#4158. Ben's PR to the upstream MAPL repository is at GEOS-ESM#4160.

The bug was in adding imports to ExtData for fields that are vectorized. When one of the two names in an ExtData2G rule for a vector, e.g. UA;VA, is contained in a substring in another item, e.g. EVAP, there would be an ESMF error that is was already added.

This fix was submitted by Ben Auer (NASA GMAO)

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
@lizziel lizziel added the bugfix Fixes a previously-reported issue label Oct 29, 2025
@lizziel lizziel added this to the GCHP_14.7.1 milestone Nov 4, 2025
This fix is being applied by GMAO in a dev version of MAPL2 beyond
MAPL 2.59 currently used in GCHP.

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
@lizziel lizziel changed the title Fix bug in parsing vector imports in ExtData2G Enable use of ExtData2G in GCHP Transport Tracers simulation Jan 5, 2026
This makes ExtData2G convert at import file 3D data that is top-down to
bottom-up prior to exporting to GCHP gridded components.

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
This is necessary to determine the vertical orientation of GEOS-Chem
processed meteorology files which are not CF-compliant.

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fixes a previously-reported issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant