-
Notifications
You must be signed in to change notification settings - Fork 33
Adapt to use GCHP outputs for IMI #337
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
Draft
1Dandan
wants to merge
84
commits into
geoschem:main
Choose a base branch
from
1Dandan:imi-gchp
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…evant lines needed for IMI
This reverts commit 1bcf973.
…tputs & use ESMF for faster calculation of overlap area
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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: Dandan Zhang
Institution: Harvard University
Describe the update
These updates are primarily for using stretched GCHP simulation for Jacobian runs in IMI, with several additional features/fixes for both IMI-GCC and IMI-GCHP. All modifications are on top of IMI v2.2.0.
Specific modifications for IMI-GCHP Jacobian runs:
As the feature of scaling grid boxes with specific value from a text file in HEMCO does not work with GCHP, as stated in a ticket of geoschem/GCHP#432, I modified the HEMCO code to apply gridded scaling factor based on the value of state vector ID at geoschem/HEMCO#330. This update of HEMCO would work for both GCC and GCHP. Thus, to minimize the differences between IMI-GCC and IMI-GCHP for synchronizing updates later, I modified the code for creating Jacobian runs in IMI-GCC as well. I tested with a Global GCC 2x2.5 and it works well. The changes would be:
ExtData.rc
andHEMCO_Config.rc
for GCHP carbon simulation forSpecific modifications for IMI-GCHP to create a grid file first:
Specific modifications for IMI-GCHP inversion (calculating Jacobian matrix):
polygon_gchp
andpolygon_tropomi
, while Shapely Polygon was used elsewhere.Fixes/features for both IMI-GCC and IMI-GCHP:
Update_prior_emis.py
at src/components/posterior_component/update_prior_emis.py was never used. Instead, this function was called from https://github.com/geoschem/integrated_methane_inversion/blob/main/src/inversion_scripts/utils.py#L552. Thus, I deleted this file.ReDoJacobian
, check if the last-day output file is valid with 24 time entries.time
dimension to be compatible with MAPL ExtData to read, including state vector, gridded perturbation scaling factor, gridded OH scaling factor, and prior emission outputs.imi_preview.py
.GC_index
during the calculation of Jacobian matrix to be used later for super observation plotting.Yaml file changes: (An example yaml file)
UseGCHP
andSTRETCH_GRID
were added toconfig.yml
. Need to add two lines of:UseGCHP: false
andSTRETCH_GRID: false
when using GCC too, as these are mandatory to know which code should be applied.OptimizeSoil
too. We need to add an entry ofOptimizeSoil: false
inconfig.yml
if not usingOptimizeSoil
.Instructions/fixes to use stretched GCHP simulations
Note: Below would be fixed in a new version of GC or MAPL ExtData, and thus you only need to make these modifications on top of the raw GCHP v14.6.2 or GCC v14.6.2.
geos-chem
repository with changes forHEMCO_Config.rc
andExtData.rc
(checking out this one), and updateHEMCO
with changes in this one.<your-gchp-repository>/src/MAPL/gridcomps/ExtData/ExtDataGridCompMod.F90
as (removing the conversion of radians to degrees) (Specific to GCHP):Further features to be added: