Fix improper handling of date timestamps and lat/lon bounds for mask files #311
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Name and Institution (Required)
Name: Melissa Sulprizio
Institution: Harvard
Describe the update
As reported in #307, mask with temporal variability were only read once per simulation. This was caused by a bug in
hco_config_mod.F90
where the year and month values for mask fields were overwritten by the lat/lon bounds of the mask file and later set to -999. To fix this, new fieldsLats
andLons
have been added to theFileData
type in hco_types_mod.F90. The lat and lon bounds are now directly stored in those 2D arrays without modifying the year and month values. The lat and lon values will now also be printed to the log file for mask files.With this fix, the
SEASONAL_RES_MASK
container in the GEOS-Chem CH4 simulation now properly lists the year and month ranges instead of -999 values. Previously, the log file listed:With this fix:
I have confirmed that the reservoir mask file in these simulations is now read and updated monthly. From the log file:
Expected changes
This is a zero-difference update with respect to full-chemistry benchmark simulations. It will impact CH4 GEOS-Chem simulations which do include mask files with temporal variability.
Related Github Issue