Skip to content

Organic Rankine Cycle Data#111

Merged
fneum merged 15 commits intoPyPSA:masterfrom
LukasFrankenQ:master
May 12, 2024
Merged

Organic Rankine Cycle Data#111
fneum merged 15 commits intoPyPSA:masterfrom
LukasFrankenQ:master

Conversation

@LukasFrankenQ
Copy link
Copy Markdown
Contributor

Adds data for Organic Rankine Cycles (ORC), and moves some data previously assigned to Enhanced Geothermal to ORC, to make things more clear.

Potentially has some overlap with data on waste or biomass, but the literature suggests this type of ORC operating around ~150 C is mostly used in a geothermal context, so it appears to warrant a separate technology. The implicit assumption of operating temperatures is stated clearly.

Also fixes a typo in the PR template.

Checklist

  • Code changes are sufficiently documented; i.e. new functions contain docstrings and further explanations may be given in doc.
  • Data source for new technologies is clearly stated.
  • Newly introduced dependencies are added to environment.yaml (if applicable).
  • A note for the release notes doc/release_notes.rst of the upcoming release is included.
  • I consent to the release of this PR's code under the GPLv3 license.

Copy link
Copy Markdown
Collaborator

@euronion euronion left a comment

Choose a reason for hiding this comment

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

Cool!

  • Why the changes to offwind-float-* technologies in the outputs/ files?
  • Since your entering the data manually, would you mind moving it into input/manual_input.csv ? Leads to less code bloating and is easier to use/track. (Maybe also the geothermal assumptions?)
  • You removed the efficiency electricity from geothermal, but this change is not represented in the output files. Do the output files need to be recreated or is there a bug somewhere?
  • Removing the geothermal efficiency electricity should also be noted in the release notes ;-)

@LukasFrankenQ
Copy link
Copy Markdown
Contributor Author

Many thanks for the review @euronion, will amend!

@LukasFrankenQ
Copy link
Copy Markdown
Contributor Author

LukasFrankenQ commented Nov 26, 2023

Hey @euronion, I moved everything to manual_input.csv, including geothermal, and think all of your points should be addressed...

... EXCEPT the changes in offshore wind, which I frankly do not understand - clearly nothing in relation to that is changed in the code. I have double checked my env, re-ran the script etc. it seems to persist. Checking the the files on github in outputs/ leaves me even more confused, as a Ctrl+F reveals they have no mention of offwind-float, which are claimed to exist by the 'Files Changed' tab of this PR.

What do you propose?

@euronion
Copy link
Copy Markdown
Collaborator

Thanks @LukasFrankenQ !

Regarding offwind-float, it looks like @p-glaum did not regenerate the output files after moving the inputs from inputs/costs_PyPSA.csv to inputs/manual_input.csv in #108 , and so the inflation adjustment was not applied. So nothing you to avoid it. I have updated the master branch, if you merge it with your PR the changes should go away.

Can you also remove the clutter in inputs/manual_input.csv from changing many XX to XX.0 numbers? Then it will be easier to review and also understand / spot mistakes in the future based on the commit log.
Thanks :)

@LukasFrankenQ
Copy link
Copy Markdown
Contributor Author

LukasFrankenQ commented Dec 18, 2023

Alright, should be all set, thanks for research with regards to the wind.

There seems to be a new issue here, should this be adjusted? Seems less straightforward which version is better...

@euronion
Copy link
Copy Markdown
Collaborator

Thanks! Much easier to review now :)

@LukasFrankenQ
Copy link
Copy Markdown
Contributor Author

Hey @euronion, I made the suggested changes, sorry for the delay! Let me know what you think :)

Copy link
Copy Markdown
Collaborator

@euronion euronion left a comment

Choose a reason for hiding this comment

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

We're getting there :)

geothermal,CO2 intensity,2020,0.12,tCO2/MWh_el,2020,"Aghahosseini, Breyer 2020: From hot rock to useful energy: A global estimate of enhanced geothermal systems potential, https://www.sciencedirect.com/science/article/pii/S0306261920312551",Likely to be improved; Average of 85 percent of global egs power plant capacity; Result of fluid circulation through rock formations
geothermal,lifetime,2020,30,years,2020,"Aghahosseini, Breyer 2020: From hot rock to useful energy: A global estimate of enhanced geothermal systems potential, https://www.sciencedirect.com/science/article/pii/S0306261920312551",
geothermal,efficiency central heat,2020,0.8,per unit,2020,"Aghahosseini, Breyer 2020: From hot rock to useful energy: A global estimate of enhanced geothermal systems potential, https://www.sciencedirect.com/science/article/pii/S0306261920312551; Breede et al. 2015: Overcoming challenges in the classification of deep geothermal potential, https://eprints.gla.ac.uk/169585/","This is a rough estimate, depends on local conditions"
geothermal,district heat-input,2020,0.8,MWh_thdh/MWh_th,2020,"Aghahosseini, Breyer 2020: From hot rock to useful energy: A global estimate of enhanced geothermal systems potential, https://www.sciencedirect.com/science/article/pii/S0306261920312551; Breede et al. 2015: Overcoming challenges in the classification of deep geothermal potential, https://eprints.gla.ac.uk/169585/","Heat-input, District Heat-output. This is an assessment of typical heat losses when heat is transmitted from the EGS plant to the DH network, This is a rough estimate, depends on local conditions"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Question: Shouldn't heat input vs. heat output be > 1?

Based on the previous 0.8 efficiency value like: 1 / 0.8 = 1.25 ?

if not, maybe I'm misunderstanding the value.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I see where the intuition is coming from, but I was convinced otherwise, and feel confident in 80% 👍

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Ok... I guess? It doesn't make any sense to me and the description in the file isn't explaining it. Would you care to indulge me? Or should I just accept and be quiet? ;-)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Heya @euronion, sorry for missing this and for going 'dude, trust me.'
I hope I get this right, but I think your intuition is summarized as such:
Say we have Q_in with temperature 150 degreesC, and it meets demand Q_out with a temperature of 90 degrees, then shouldnt that happen at an efficiency of > 1.. Is that the intuition? Otherwise we are talking about different things.
What this intuition misses is that the temperature is already priced into the power (roughly mass flow times temperature). So when thinking about situation where Q_in = Q_out, but they have a 60 degreeC temperature difference what we are really talking about is the hotter flow having a much smaller mass flow and us mixing in cooler liquid to accommodate the properties required at Q_out... Does that help? Maybe it wasnt the problem, does not feel very complex but mass flow is what was missing in my intuition.

@LukasFrankenQ
Copy link
Copy Markdown
Contributor Author

Hi @euronion, apologies for the delay, worked in the feedback! Let me know if there is anything else 🙌

@fneum fneum merged commit 838db9e into PyPSA:master May 12, 2024
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