Skip to content

Conversation

@lizziel
Copy link
Contributor

@lizziel lizziel commented Dec 3, 2025

Name and Institution (Required)

Name: Lizzie Lundgren
Institution: Harvard University

Describe the update

This PR incorporates new entries in GCHP config file GCHP.rc that are brought in with a companion PR in the GEOS-Chem repository (geoschem/geos-chem#3131). This fixes a problem vertically flipping met-fields when using GEOS-FP C720 files (top-down) with GEOS-FP processed 0.25x0.625 files (bottom-up).

Expected changes

This update does not impact benchmarking. It is zero diff except for the specific case of using GEOS-FP C720 files with GEOS-FP processed fields.

Reference(s)

None

Related Github Issue

#527
Companion PR required: geoschem/geos-chem#3131

@lizziel lizziel added topic: Input Data Related to input data category: Bug Fix Fixes a previously-reported issue labels Dec 3, 2025
@lizziel lizziel changed the base branch from main to dev/14.7.0 December 3, 2025 22:11
@lizziel lizziel added this to the 14.7.0 milestone Dec 3, 2025
@lizziel lizziel added the no-diff-to-benchmark This update will not change the results of fullchem benchmark simulations label Dec 3, 2025
@lizziel lizziel marked this pull request as draft December 3, 2025 22:12
@lizziel
Copy link
Contributor Author

lizziel commented Dec 3, 2025

This PR is done but has not yet been tested.

@lizziel lizziel marked this pull request as ready for review December 4, 2025 18:58
Copy link
Contributor

@yantosca yantosca left a comment

Choose a reason for hiding this comment

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

Thanks @lizziel. This looks good to merge.

…issue

This update requires a submodule update for GEOS-Chem. The issue only
affected usage of GEOS-FP C720 files (top-down) with processed met
fields (bottom-up).

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
…s flux

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
@lizziel lizziel force-pushed the bugfix/fix_vertical_flipping_C720_fields_with_processed_met branch from 9e46d1b to 532149f Compare December 10, 2025 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: Bug Fix Fixes a previously-reported issue no-diff-to-benchmark This update will not change the results of fullchem benchmark simulations topic: Input Data Related to input data

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants