Skip to content

Conversation

@acollow
Copy link
Contributor

@acollow acollow commented Jun 4, 2025

Entries for energy, non-energy, and shipping emissions have been removed from the ExtData.yaml files for carbon, sulfate, and nitrate as these enter through HEMCO. This change must be pulled in at the same time as a PR in GEOSChem_GridComp. The changes are not fully 0 diff however the difference in the emissions when piped through HEMCO instead of entering through GOCART is within the range of uncertainty and below the non-conservative nature of the daily temporal interpolation. This PR must be take alongside GEOS-ESM/GEOSchem_GridComp#300 to ensure emissions are not double counted or missing.

@acollow acollow requested a review from a team as a code owner June 4, 2025 17:36
@acollow acollow added 0 diff trivial The changes in this pull request are trivially zero-diff (documentation, build failure, &c.) Contingent - DNA Do Not Approve (DNA). These changes are contingent on other PRs labels Jun 4, 2025
@github-actions
Copy link

github-actions bot commented Jun 4, 2025

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: 0 diff trivial, Contingent - DNA

2 similar comments
@github-actions
Copy link

github-actions bot commented Jun 4, 2025

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: 0 diff trivial, Contingent - DNA

@github-actions
Copy link

github-actions bot commented Jun 4, 2025

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: 0 diff trivial, Contingent - DNA

@acollow acollow removed the Contingent - DNA Do Not Approve (DNA). These changes are contingent on other PRs label Jun 4, 2025
@pcolarco pcolarco added Non 0-diff The changes in this pull request are non-zero-diff and removed 0 diff trivial The changes in this pull request are trivially zero-diff (documentation, build failure, &c.) labels Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Non 0-diff The changes in this pull request are non-zero-diff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants