-
Notifications
You must be signed in to change notification settings - Fork 170
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
Add CEDS 0.1 x 0.1 degree emissions #2171
Conversation
run/GCClassic/HEMCO_Config.rc.templates/HEMCO_Config.rc.fullchem
Outdated
Show resolved
Hide resolved
@1Dandan @Ruijun-Dang: Just wanted to tag #1745 (comment) on this PR. Will await your feedback before proceeding. |
Hi @yantosca @msulprizio, the seasonality problem as mentioned in #1745 should be fixed in the new version of v2024-06. File names remain the same as those in v2023-04, and thus all v2023-04 can be replaced by v2024-06 for HEMCO_Config.rc and ExtData.rc templates. Let me know if you want other inputs from me. |
run/GCClassic/HEMCO_Config.rc.templates/HEMCO_Config.rc.fullchem run/GCHP/HEMCO_Config.rc.templates/HEMCO_Config.rc.fullchem - Added entries for CEDS 0.1 x 0.1 degree emissions. NOTE: ExtData.rc.fullchem will be updated later. CHANGELOG.md - Updated accordingly Signed-off-by: Bob Yantosca <[email protected]>
run/GCClassic/HEMCO_Config.rc.templates/HEMCO_Config.rc.fullchem - CEDS01 comment update "1750-2019" -> "1980-2019" - Also updated header comments run/GCClassic/HEMCO_Config.rc.templates/HEMCO_Config.rc.aerosol run/GCClassic/HEMCO_Config.rc.templates/HEMCO_Config.rc.carbon run/GCClassic/HEMCO_Config.rc.templates/HEMCO_Config.rc.TransportTracers run/GCHP/HEMCO_Config.rc.templates/HEMCO_Config.rc.carbon run/GCHP/HEMCO_Config.rc.templates/HEMCO_Config.rc.TransportTracers run/GCHP/HEMCO_Config.rc.templates/HEMCO_Config.rc.fullchem run/GCHP/ExtData.rc.templates/ExtData.rc.carbon run/GCHP/ExtData.rc.templates/ExtData.rc.fullchem run/GCHP/ExtData.rc.templates/ExtData.rc.TransportTracers - Added entries for CEDS_01x01 as the default - Now make CEDSv2 an option Signed-off-by: Bob Yantosca <[email protected]>
run/GCClassic/HEMCO_Config.rc.templates/HEMCO_Config.rc.fullchem run/GCHP/HEMCO_Config.rc.templates/HEMCO_Config.rc.fullchem - For container "CEDS01_TOLU_WST", added scale factor=26, cat=1, hier=5. This had been omitted due to a cut-n-paste error. Signed-off-by: Bob Yantosca <[email protected]>
run/GCHP/ExtData.rc.TransportTracers - Fixed typo: "shoul'd" -> "should Signed-off-by: Bob Yantosca <[email protected]>
This rebase merge brings PR #2171 (CEDS 0.1 x 0.1 emissions) atop PR #2352 (ALK4 & R4N2 chemistry updates) Signed-off-by: Bob Yantosca <[email protected]>
5b9946e
to
34f8f67
Compare
Hi @yantosca, I noticed that the CEDS 0.1 version is still |
Thanks @1Dandan, yes I just rebased the existing code atop of dev/14.5.0. I'll push another commit to update the directory paths. |
Merged 14.6.0-alpha.2 into this branch and am running integration tests. |
After merging 14.6.0-alpha.2 into this branch, all GEOS-Chem Classic integration tests passed: ==============================================================================
GEOS-Chem Classic: Execution Test Results
CodeDir : b7e6523 GEOS-Chem update: Merge PR #2683 (Restore clobbered TMB updates)
GEOS-Chem : b474a7efb Fix typos in HEMCO_Config.rc.carbon for GCClassic & GCHP
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: 66516782
==============================================================================
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%%% All execution tests passed! %%%
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% |
After merging 14.6.0-alpha.2 into this branch, all GCHP integration tests passed: ==============================================================================
GCHP: Execution Test Results
CodeDir : 3089c38 GEOS-Chem update: Merge PR #2683 (Restore clobbered TMB updates)
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 : 98b088078 Merge 14.6.0-alpha.2 into feature/ceds-0.1-degree
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: 66509840
==============================================================================
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%%% All execution tests passed! %%%
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% |
@yantosca There also seem to be some fixes for the GCHP carbon simulation in this PR (see acf8ff8) that are unrelated to the CEDS update. Do you know if this impacts the GCHP carbon simulation or if they're zero-diff updates? Should we remove those updates from this branch/PR and make a separate PR for them? |
run/GCClassic/HEMCO_Config.rc.templates/HEMCO_Config.rc.carbon run/GCClassic/HEMCO_Config.rc.templates/HEMCO_Config.rc.tagCO run/GCHP/HEMCO_Config.rc.templates/HEMCO_Config.rc.carbon - Change CEDS_01x01 from ${RUNDIR_USE_CEDS} to true - Change CEDS_01x01_SHIP from ${RUNDIR_USE_CEDS} to true run/GCHP/ExtData/ExtData.rc.templates/ExtData.rc.TransportTracers run/GCHP/ExtData/ExtData.rc.templates/ExtData.rc.fullchem - Updated comments "CEDSv2" -> "CEDS_01x01" Signed-off-by: Bob Yantosca <[email protected]>
run/GCHP/ExtData.rc.templates/ExtData.rc.carbon - Restored modifications that were made specifically in the feature/ceds-0.1-degree branch back to what they were in dev/14.6.0. Signed-off-by: Bob Yantosca <[email protected]>
@msulprizio: I restored the updates in question to what they were in the dev/14.6.0 branch. This was done in commit 6693d46 |
run/GCHP/ExtData.rc.templates/ExtData.rc.TransportTracers - Fixed typo in entries for CEDS_CO_25 filenames ("CEDS_CO2_0.1x0.1_%y4.nc" -> "CEDS_CO_0.1x0.1_%y4.nc") run/GCHP/ExtData.rc.templates/ExtData.rc.carbon - Removed RCP and ICOADS_SHIP entries (leftover from merge up) Signed-off-by: Bob Yantosca <[email protected]>
13703fe
to
6de9ca5
Compare
These changes were originally introduced by Bob Yantosca (@yantosca) in commit acf8ff8 in PR #2171 (CEDS 0.1x0.1 emissions) but they should be in their own PR. The fixes have now been added on top of 14.5.1 which also included many fixes to ExtData.carbon. Fixes here include: - Change climatology data entries from e.g "2009 Y" to "Y Y F2009-..." - For CO2_WEEKLY, use %d2 to ensure daily update Signed-off-by: Melissa Sulprizio <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @yantosca. All of my comments have been addressed. Good to merge!
I have merged this into the dev/14.6.0 branch (locally) and am running integration tests. Stay tuned for results. |
CHANGELOG.md - Updated based on comments by @lizziel on PR #2171 Signed-off-by: Bob Yantosca <[email protected]>
run/GCHP/ExtData.rc.templates/ExtData.rc.carbon - Restored several entries for the CEDS_01x01 CO emissions, which appear to have been clobbered during a merge. Signed-off-by: Bob Yantosca <[email protected]>
GEOS-Chem Classic integration tests all passed after having made the fixes discussed above: ==============================================================================
GEOS-Chem Classic: Execution Test Results
CodeDir : b7e6523 GEOS-Chem update: Merge PR #2683 (Restore clobbered TMB updates)
GEOS-Chem : c5a667abe Merge PR #2683 (Restore updates for TMB emissions)
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: 66868886
==============================================================================
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%%% All execution tests passed! %%%
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% |
After having made all the fixes discussed above, GCHP integration tests also passed: ==============================================================================
GCHP: Execution Test Results
CodeDir : 3089c38 GEOS-Chem update: Merge PR #2683 (Restore clobbered TMB updates)
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 : 4e663f76a Bug fix: restore missing CEDS_CO entries in ExtData.rc.carbon
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: 66890631
==============================================================================
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%%% All execution tests passed! %%%
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%% |
This merge brings PR # (Fix merge conflict that emitted CEDS C6H14 emissions into ALK4 instead of ALK6; Also emit C6H14 into ALK6 in CMIP6 and HTAP3 inventories, by @yantosca) into the GEOS-Chem 14.6.0 development stream. This PR does the following: 1. Restores updates to HEMCO_Config.rc.fullchem templates that were clobbered in the merge of PR #2171 2. Emits C6H14 (hexanes) from CMIP6 and HTAPv3 inventories into ALK6, to be consistent with what we alreadyd do for CEDS. Signed-off-by: Bob Yantosca <[email protected]>
Name and Institution (Required)
Name: Bob Yantosca
Institution: Harvard + GCST
Confirm you have reviewed the following documentation
Describe the update
This is the companion PR to #1745.
Expected changes
As described in #1745
Related Github Issue(s)
Tagging @msulprizio