Skip to content

Allow absence of offshore wind from sector-coupled networks#944

Merged
fneum merged 5 commits intoPyPSA:masterfrom
koen-vg:no-offwind-fix
Feb 21, 2024
Merged

Allow absence of offshore wind from sector-coupled networks#944
fneum merged 5 commits intoPyPSA:masterfrom
koen-vg:no-offwind-fix

Conversation

@koen-vg
Copy link
Copy Markdown
Contributor

@koen-vg koen-vg commented Feb 20, 2024

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.

@fneum
Copy link
Copy Markdown
Member

fneum commented Feb 20, 2024

Great! I just merged the new scenario management, which has quite some changes and this branch needs to be adjusted to that. Otherwise the changes look good!

What you'd need to do is to mirror the following lines of code:

def input_profile_tech(w):
return {
f"profile_{tech}": resources(f"profile_{tech}.nc")
for tech in config_provider("electricity", "renewable_carriers")(w)
}

then

input:
unpack(input_profile_tech),
unpack(input_conventional),

The unpack() function allows the use of the wildcard-aware config_provider().

@koen-vg
Copy link
Copy Markdown
Contributor Author

koen-vg commented Feb 20, 2024

Okay thanks, I'll get around to it shortly!

@koen-vg
Copy link
Copy Markdown
Contributor Author

koen-vg commented Feb 20, 2024

Alright the first time I didn't test well enough but this time it should all work. I have it from good sources that this will make fellow energy system modellers in Austria quite happy!

@fneum fneum merged commit 9b94d9d into PyPSA:master Feb 21, 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.

2 participants