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

Eu security reimplementation #402

Merged
merged 50 commits into from
Jul 21, 2022
Merged

Eu security reimplementation #402

merged 50 commits into from
Jul 21, 2022

Conversation

pz-max
Copy link
Member

@pz-max pz-max commented Jul 4, 2022

Closes #395 #390

Changes proposed in this Pull Request

In summary the PR adds:

  • marginal cost parameter sweeps through wildcards
  • gas limit constraint
  • harmonized aggregation strategies
  • make carriers easy to read in config & easier to define
  • move snakemake inputs out of function to improve debugging experience

In detail:

  • Add function to add global constraint on use of gas in :mod:prepare_network. This can be activated with electricity: gaslimit: given in MWh.

  • A new section conventional was added to the config file. This section contains configurations for conventional carriers.

  • The powerplants that have been shut down before 2021 are filtered out.

  • The inclusion of renewable carriers is now specified in the config entry renewable_carriers. Before this was done by commenting/uncommenting sub-sections in the renewable config section.

  • Now, all carriers that should be extendable have to be listed in the config entry extendable_carriers. Before, renewable carriers were always set to be extendable. For backwards compatibility, the workflow is still looking at the listed carriers under the renewable key. In the future, all of them have to be listed under extendable_carriers.

  • It is now possible to set conventional power plants as extendable by adding them to the list of extendable Generator carriers in the config.

  • Listing conventional carriers in extendable_carriers but not in conventional_carriers, sets the corresponding conventional power plants as extendable without a lower capacity bound of today's capacities.

  • Now, conventional carriers have an assigned capital cost by default.

  • The build_year and lifetime column are now defined for conventional power plants.

Checklist

  • I tested my contribution locally and it seems to work fine.
  • Code and workflow changes are sufficiently documented.
  • Changes in configuration options are added in all of config.default.yaml, config.tutorial.yaml, and test/config.test1.yaml.
  • Changes in configuration options are also documented in doc/configtables/*.csv and line references are adjusted in doc/configuration.rst and doc/tutorial.rst.
  • A note for the release notes doc/release_notes.rst is amended in the format of previous release notes, including reference to the requested PR.

pz-max and others added 30 commits July 4, 2022 14:43
@pz-max
Copy link
Member Author

pz-max commented Jul 6, 2022

@davide-f ready for review

Generator: [solar, onwind, offwind-ac, offwind-dc, OCGT]
StorageUnit: [] # battery, H2
Store: [battery, H2]
Link: [AC, DC]
Copy link
Member

Choose a reason for hiding this comment

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

We have to understand the behavior of this PR combined with the one with HVDC lines implemented as dc lines

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, once @ekatef PR is merged we have to revise quite a bit here
I would take care of it & potentially discuss changes with @ekatef

Btw, I think here the word link refers rather to what "network" is included

scripts/prepare_network.py Show resolved Hide resolved
config.default.yaml Show resolved Hide resolved
scripts/prepare_network.py Show resolved Hide resolved
Copy link
Member

@davide-f davide-f left a comment

Choose a reason for hiding this comment

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

@pz-max a small comment and once answered or addressed we can proceed

@pz-max
Copy link
Member Author

pz-max commented Jul 21, 2022

@pz-max a small comment and once answered or addressed we can proceed

Addressed the issue hopefully above. Thank you very much for reviewing. Excellent job!

Copy link
Member

@davide-f davide-f left a comment

Choose a reason for hiding this comment

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

Awesome! Ready to go :)

@davide-f davide-f merged commit e46f324 into pypsa-meets-earth:main Jul 21, 2022
pz-max pushed a commit that referenced this pull request Sep 24, 2022
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.

Update PyPSA-Africa according major PyPSA-Eur changes
2 participants