Skip to content
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

Merged
merged 16 commits into from
Jan 17, 2025
Merged

Conversation

yantosca
Copy link
Contributor

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

@yantosca yantosca self-assigned this Feb 29, 2024
@yantosca yantosca changed the title Edit HEMCO_Config.rc.fullchem for CEDS 0.1 degree emisisons Add CEDS 0.1 x 0.1 degree emissions Feb 29, 2024
@yantosca yantosca added category: Feature Request New feature or request topic: Input Data Related to input data topic: Configuration Files Related to GEOS-Chem configuration files labels Feb 29, 2024
@yantosca yantosca added this to the 14.4.0 milestone Feb 29, 2024
@yantosca yantosca added deferred Issues/PRs that we cannot work on right away help needed: Request Input From Community We need GC community to provide a fix/update and removed deferred Issues/PRs that we cannot work on right away labels Mar 20, 2024
@yantosca yantosca linked an issue Mar 20, 2024 that may be closed by this pull request
@yantosca
Copy link
Contributor Author

@1Dandan @Ruijun-Dang: Just wanted to tag #1745 (comment) on this PR. Will await your feedback before proceeding.

@yantosca yantosca removed this from the 14.4.0 milestone Mar 20, 2024
@yantosca yantosca added the deferred Issues/PRs that we cannot work on right away label Mar 22, 2024
@yantosca yantosca deleted the branch dev/14.6.0 May 30, 2024 18:55
@yantosca yantosca closed this May 30, 2024
@msulprizio msulprizio reopened this May 30, 2024
@msulprizio msulprizio changed the base branch from dev/14.4.0 to main May 30, 2024 19:16
@1Dandan
Copy link
Contributor

1Dandan commented Jun 21, 2024

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]>
@yantosca yantosca force-pushed the feature/ceds-0.1-degree branch from 5b9946e to 34f8f67 Compare August 12, 2024 20:07
@yantosca yantosca changed the base branch from main to dev/14.5.0 August 12, 2024 20:08
@1Dandan
Copy link
Contributor

1Dandan commented Aug 12, 2024

Hi @yantosca, I noticed that the CEDS 0.1 version is still v2023-04, which needs to be changed to v2024-06.
Also, I deleted all files at CEDS/v2023-04 at http://geoschemdata.wustl.edu/ExtData/HEMCO/CEDS/ to avoid any confusion as v2024-06 should be used instead.

@yantosca
Copy link
Contributor Author

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.

@yantosca
Copy link
Contributor Author

Merged 14.6.0-alpha.2 into this branch and am running integration tests.

@yantosca
Copy link
Contributor Author

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!  %%%
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%

@yantosca
Copy link
Contributor Author

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 yantosca marked this pull request as ready for review January 16, 2025 16:01
@msulprizio
Copy link
Contributor

@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]>
@yantosca
Copy link
Contributor Author

@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?

@msulprizio: I restored the updates in question to what they were in the dev/14.6.0 branch. This was done in commit 6693d46

@yantosca yantosca requested a review from msulprizio January 16, 2025 17:52
run/GCHP/ExtData.rc.templates/ExtData.rc.TransportTracers Outdated Show resolved Hide resolved
run/GCHP/ExtData.rc.templates/ExtData.rc.carbon Outdated Show resolved Hide resolved
run/GCHP/ExtData.rc.templates/ExtData.rc.carbon Outdated Show resolved Hide resolved
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]>
@yantosca yantosca force-pushed the feature/ceds-0.1-degree branch from 13703fe to 6de9ca5 Compare January 16, 2025 18:39
@yantosca yantosca requested a review from msulprizio January 16, 2025 18:40
msulprizio added a commit that referenced this pull request Jan 16, 2025
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]>
Copy link
Contributor

@msulprizio msulprizio left a 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!

@yantosca
Copy link
Contributor Author

I have merged this into the dev/14.6.0 branch (locally) and am running integration tests. Stay tuned for results.

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md
- Updated based on comments by @lizziel on PR #2171

Signed-off-by: Bob Yantosca <[email protected]>
@yantosca yantosca requested a review from lizziel January 17, 2025 16:33
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]>
@yantosca yantosca requested a review from lizziel January 17, 2025 18:58
@yantosca
Copy link
Contributor Author

@lizziel: Missing entries in the ExtData.rc.carbon file was causing the integration tests to fail. These were probably clobbered in the merge. They've been restored in 4e663f7. I will ask for a re-review if the GCHP integration tests pass.

@yantosca
Copy link
Contributor Author

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!  %%%
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%

@yantosca
Copy link
Contributor Author

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!  %%%
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%

@yantosca yantosca merged commit 9aeb0da into dev/14.6.0 Jan 17, 2025
@yantosca yantosca deleted the feature/ceds-0.1-degree branch January 17, 2025 20:55
yantosca added a commit that referenced this pull request Jan 22, 2025
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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Feature Request New feature or request topic: Configuration Files Related to GEOS-Chem configuration files topic: Emissions Related to emissions inventories used in GEOS-Chem topic: Input Data Related to input data
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adding CEDS at 0.1 degree as a global emission inventory
7 participants