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

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

Merged
merged 5 commits into from
Feb 21, 2024

Conversation

koen-vg
Copy link
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
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
Contributor Author

koen-vg commented Feb 20, 2024

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

@koen-vg
Copy link
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
6 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.

2 participants