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

Read carbon capture data from new DEA industry sheets #20

Merged
merged 7 commits into from
Dec 11, 2020
Merged

Conversation

nworbmot
Copy link
Member

@nworbmot nworbmot commented Dec 7, 2020

Read in for direct air capture and post combustion capture on a small biomass CHP and a cement kiln.

nworbmot and others added 3 commits December 7, 2020 18:37
Read in for direct air capture and post combustion capture on a small
biomass CHP and a cement kiln.
Copy link
Collaborator

@lisazeyen lisazeyen left a comment

Choose a reason for hiding this comment

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

We can merge this branch. In the long-term I would like to

(1) remove the extra function add_carbon_capture(data) and rather include processing the data in the previous steps
(2) add as well the costs for gas CHPs CCS/ carbon capture from p.86 DEA version 0009
(3) than remove the function add_costs_ccs(costs) which currently adds costs for CCS from DIW

@@ -929,6 +926,31 @@ def add_gas_storage(data):

return data

def add_carbon_capture(data, tech_data):
Copy link
Collaborator

Choose a reason for hiding this comment

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

it would be better to include the technology in the previous functions to handle the data instead of writing a single function for carbon capture

@@ -1054,8 +1079,6 @@ def rename_ISE(costs_ISE):
costs = add_co2_intensity(costs)
# CCS
costs = add_costs_ccs(costs)
Copy link
Collaborator

Choose a reason for hiding this comment

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

is this function still needed or can it be removed? we should use the CCS costs for gas CHPs from DEA p.86

@@ -82,6 +82,9 @@
'biogas upgrading': '82 Biogas, upgrading',
'battery': '180 Lithium Ion Battery',
'electrolysis': '88 Alkaline Electrolyser',
'direct air capture' : '403.a Direct air capture',
'biomass CHP capture' : '401.a Post comb - small CHP',
'cement capture' : '401.c Post comb - Cement kiln',
Copy link
Collaborator

Choose a reason for hiding this comment

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

we can also check if they have gas CHP capture in DEA

Copy link
Member

@martavp martavp Dec 9, 2020

Choose a reason for hiding this comment

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

Yes, it does, in section '07 CO2 Capture and Storage'

(I just realized you mentioned the same in another comment)

I.e. if only carbon capture is involved, but second step of
sequestration or usage is not yet clear, rename CCS as CC.
@martavp
Copy link
Member

martavp commented Dec 9, 2020

I agree. We can merge this branch, but @nworbmot maybe want to address @lisazeyen before merging

@nworbmot
Copy link
Member Author

nworbmot commented Dec 9, 2020

Hi both, yes, I agree with @lisazeyen's points. I'm just tidying up the scripts a bit now. I have removed the redundant function "add_costs_ccs", i.e. point (3). That doesn't address points (1) and (2) but it's a start. Will push soon, then merge, then do a release to coincide with PyPSA-Eur-Sec v0.4.0.

nworbmot and others added 2 commits December 9, 2020 18:32
This has been replaced by DEA "cement capture" assumptions.
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.

None yet

3 participants