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

Move other CDR to Emissions|CO2|Other Capture and Removal #434

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

laurinks
Copy link
Contributor

@laurinks laurinks commented Jan 21, 2025

Purpose of this PR

  1. Move DACCS and EW from Emissions|CO2|Other to Emissions|CO2|Other Capture and Removal
  2. Also add OAE to Emissions|CO2|Other Capture and Removal (no OAE in last submission, but as preparation for future submissions)

@merfort : Looking at CDR across models, I realized that we map DACCS and EW to Other, not to Other Capture and Removal. Other teams map to Other Capture and Removal. Should this be corrected for future submissions or is there a good reason for this in light of the emissions harmonization process?

Todo:

  • Build Library

Checklist:

  • I did not use Excel to open csv files or checked that no side-effects occur (changed values, many new quotation marks, …)
  • I added any renamed piam_variable to renamed_piam_variables.csv to guarantee backwards compatibility.

It is recommended to have a look at the tutorial before submission.

@laurinks
Copy link
Contributor Author

There are a few reporting issues listed in the ScenarioMIP CDR sheet. Would be good to correct them with this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant