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

Add floating wind technology #773

Merged
merged 13 commits into from
May 13, 2024
Merged

Add floating wind technology #773

merged 13 commits into from
May 13, 2024

Conversation

p-glaum
Copy link
Contributor

@p-glaum p-glaum commented Nov 6, 2023

Closes #36 but changed it to water depth below 60m

Changes proposed in this Pull Request

Add floating wind technology to PyPSA-Eur framework. Need to first update the technology data master with PR in PyPSA/technology-data#108.

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.

@p-glaum p-glaum marked this pull request as draft November 6, 2023 15:38
@fneum fneum mentioned this pull request Jan 2, 2024
9 tasks
@koen-vg
Copy link
Contributor

koen-vg commented May 10, 2024

Hi! Any chance of this getting merged in the near future? I've been using this branch in two different projects already and it seems to work well! Here's my version with the latest changes from the master branch merged: https://github.com/koen-vg/pypsa-eur/tree/implement-floating

I guess I have vested interests in this branch since floating offshore wind is disproportionally relevant in Norway, where there's not that much space for bottom-fixed offshore wind.

I guess my only comment is that I believe the costs in technology-data to be on the optimistic side, but this is so hard to tell anyway. If someone wants to do serious work on floating offshore wind in pypsa-eur, they should look into the costs anyway.

@p-glaum p-glaum marked this pull request as ready for review May 10, 2024 15:38
@p-glaum
Copy link
Contributor Author

p-glaum commented May 10, 2024

Hi! Any chance of this getting merged in the near future? I've been using this branch in two different projects already and it seems to work well! Here's my version with the latest changes from the master branch merged: https://github.com/koen-vg/pypsa-eur/tree/implement-floating

I guess I have vested interests in this branch since floating offshore wind is disproportionally relevant in Norway, where there's not that much space for bottom-fixed offshore wind.

I guess my only comment is that I believe the costs in technology-data to be on the optimistic side, but this is so hard to tell anyway. If someone wants to do serious work on floating offshore wind in pypsa-eur, they should look into the costs anyway.

Hi @koen-vg,

The CI always failed, and I believe that since we had many other PRs merged in the meantime, this PR was somewhat forgotten. Thanks for testing it, and I think it will be merged soon, as the CI has now passed. Could I ask what your cost assumption for floating wind is and from which source? Perhaps we can adjust those in the technology data as well.

@fneum fneum added this to the v0.11.0 milestone May 12, 2024
@koen-vg
Copy link
Contributor

koen-vg commented May 13, 2024

On costs for floating offshore wind, I've previously used estimates given by the Norwegian Water Resource and Energy Directorate. They produced some numbers back in 2021 which can be found here. However, it seems a little bit tricky to track down precisely how they arrived at these numbers, and at the same time they are saying they will produce updated cost estimates this year based on ongoing work with floating offshore wind.

All in all I think it's probably better to wait for these new numbers (whenever they arrive), and I'd be fine with the current costs being used for now.

Copy link
Member

@fneum fneum left a comment

Choose a reason for hiding this comment

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

LGTM

config/config.default.yaml Outdated Show resolved Hide resolved
@fneum fneum merged commit 430b95d into PyPSA:master May 13, 2024
0 of 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.

Add floating wind turbines for sea deeper than 50 metres
3 participants