Fix merge conflict that emitted CEDS C6H14 emissions into ALK4 instead of ALK6; Also emit C6H14 into ALK6 in CMIP6 and HTAP3 inventories #2696
Conversation
This file contains 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
run/GCClassic/HEMCO_Config.rc.templates/HEMCO_Config.rc.fullchem run/GCHP/HEMCO_Config.rc.templates/HEMCO_Config.rc.fullchem - Add C6H14 (hexane) emissions into the ALK6 species. This was done for CEDS in 14.5.0 when the ALK6 species was added. But the emissions entries for C6H14 in the other inventories (CMIP6, HTAPv3, etc) were not adjusted at that time. This is now fixed. - ALSO NOTE: A merge conflict clobbered the CEDS C6H14 emissions so these emissions went into ALK4 instead. This caused large differences when comparing to 14.6.0-alpha.2 This has now been fixed. Signed-off-by: Bob Yantosca <[email protected]>
yantosca
added
topic: Configuration Files
|
Integration tests are running. |
|
All GEOS-Chem Classic integration tests passed: ==============================================================================
GEOS-Chem Classic: Execution Test Results
CodeDir : ed0c344 GEOS-Chem update: Merge PR #2181 (CEDS 0.1 x 0.1 emissions)
GEOS-Chem : 5b21ce3d5 Add C6H14 emissions into ALK6 species for CEDS & other inventories
HEMCO : 0ae25d2 HEMCO 3.10.1 release
Cloud-J : f8a2b7f Update version number for 8.0.1 release
HETP : 2a99b24 Merge pull request #2 from geoschem/bugfix/initialize_local_variables
Using 24 OpenMP threads
Number of execution tests: 30
Submitted as SLURM job: 301464
==============================================================================
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%%% All execution tests passed! %%%
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%All GCHP integration tests passed as well: ==============================================================================
GCHP: Execution Test Results
CodeDir : c6fd08b GEOS-Chem update: Merge PR #2181 (CEDS 0.1 x 0.1 emissions)
MAPL : 9ad63ae Merge PR #37 containing update to vertically flip imports with dimensionless pressure proxy lev coordinates
GMAO_Shared : 4ddb3ec Merge pull request #2 from geoschem/feature/mapl-upgrade
ESMA_cmake : ad5deba Added ecbuild as a submodule of ESMA_cmake
gFTL-shared : 4b82492 Merge branch 'upstream_v1.5.0' into feature/v1.5.0
FMS : 259759d Merge pull request #3 from geoschem/feature/update_gmao_libs
FVdycoreCubed : af42462 Merge PR #8 (Add PLEadv diagnostic for offline advection in GCHP)
geos-chem : 5b21ce3d5 Add C6H14 emissions into ALK6 species for CEDS & other inventories
HEMCO : 0ae25d2 HEMCO 3.10.1 release
yaFyaml : 19afe50 Merge branch 'upstream_v1.0.4' into feature/v1.0.4
pFlogger : 2c4b724 Merge branch 'upstream_v1.9.1' into feature/v1.9.1
Cloud-J : f8a2b7f Update version number for 8.0.1 release
HETP : 2a99b24 Merge pull request #2 from geoschem/bugfix/initialize_local_variables
Number of execution tests: 12
Submitted as SLURM job: 301673
==============================================================================
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%%% All execution tests passed! %%%
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% |
msulprizio
approved these changes
Jan 22, 2025
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: Bob Yantosca
Institution: Harvard + GCST
Describe the update
This PR does the following:
NOTE: The ALK6 species was added in the Travis et al 2024 emissions updates from @kelvinhb for CEDS only, but we never made the corresponding edits in HEMCO_Config.rc for the CMIP6 and HTAPv3 inventories. This is now resolved.
Expected changes
This will fix an issue where ALK4 recieved almost double the emissions that it should have and ALK6 received about half the emissions that it should have.
Related Github Issue