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

Enable scneario management for different configs #413

Merged
merged 6 commits into from
Sep 13, 2022
Merged

Conversation

FabianHofmann
Copy link
Contributor

@FabianHofmann FabianHofmann commented Sep 8, 2022

From the release notes:

The software now supports running the workflow with different settings within the same directory. A new config section run was created that specifies under which scenario name the created resources, networks and results should be stored. If name is not specified, the workflow uses the default paths. The entry dedicated_cutout specifies whether the run should use cutouts from the default root directory and use run-specific cutouts.

@fneum @p-glaum any suggestion on how to rename dedicated_cutouts? No so happy with this key...

Note this is a non-breaking change.

Checklist

  • I tested my contribution locally and it seems to work fine.
  • Code and workflow changes are sufficiently documented.
  • 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.

FabianHofmann and others added 2 commits September 8, 2022 20:32
Co-authored-by: Philipp Glaum <95913147+p-glaum@users.noreply.github.com>
@FabianHofmann
Copy link
Contributor Author

@fneum any objection to merge this?

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.

Nice! Especially, since it keeps paths as they were if no run-name is specified!

Just a few minor comments:

Could you document new run options also at

https://github.com/PyPSA/pypsa-eur/blob/master/doc/configtables/toplevel.csv

FabianHofmann and others added 3 commits September 13, 2022 09:47
Co-authored-by: Fabian Neumann <fabian.neumann@outlook.de>
Co-authored-by: Fabian Neumann <fabian.neumann@outlook.de>
@fneum fneum merged commit aac7f66 into master Sep 13, 2022
@fneum fneum deleted the scenario-management branch September 13, 2022 13:16
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