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

Excluding Emissions from Sector Ratios Development #1154

Merged
merged 3 commits into from
Jul 24, 2024

Conversation

toniseibold
Copy link
Contributor

@toniseibold toniseibold commented Jul 15, 2024

Closes (PyPSA/pypsa-ariadne#124)

Changes proposed in this Pull Request

Currently a linear interpolation between current sector ratios and future sector ratios is assumed. However, in the current sector ratios, process emission and process emission from feedstock do not exist. Together with the config sector_ratios_fraction_future, this leads to 0 emissions in the industry sector in 2020 and a linear growth until 2045.

I assumed that the emissions in the industry currently are not going to magically disappear in 2020 and increase until 2045 so I set it to be constant.

image

Checklist

  • I tested my contribution locally and it seems to work fine.
  • Code and workflow changes are sufficiently documented.
  • Changed dependencies are added to envs/environment.yaml.
  • Changes in configuration options are added in all of config.default.yaml.
  • Changes in configuration options are also documented in doc/configtables/*.csv.
  • A release note doc/release_notes.rst is added.

@toniseibold toniseibold requested review from fneum and lindnemi July 15, 2024 11:17
Copy link
Contributor

@lindnemi lindnemi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you paste Emissions|CO2|Industrial Processes?

Before this PR it looked like this:

Emissions-CO2-Industrial-Processes

I also wonder why process emissions is missing in today_ratios?

@toniseibold
Copy link
Contributor Author

Emissions|CO2|Industrial Processes|Mt CO2/yr| 34.5449093478852 | 31.7975013193079 | 3.7944849581216 | 3.96369796528107 | 4.17226169369013 | 4.4065174844019

@toniseibold toniseibold marked this pull request as ready for review July 16, 2024 13:19
@lindnemi
Copy link
Contributor

Looks good to me. @fneum should we use this PR or (partly) revert the offending fillna?

Just a note: The model seems to switch very quickly to process emissions CC in 2030. Maybe we should limit the PE CC build out speed

@fneum fneum changed the base branch from ariadne2 to master July 24, 2024 16:15
@fneum fneum changed the base branch from master to ariadne2 July 24, 2024 16:17
@fneum
Copy link
Member

fneum commented Jul 24, 2024

cherry-picked in master via 9b78319

@fneum fneum merged commit 2ccb580 into ariadne2 Jul 24, 2024
2 checks passed
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.

3 participants