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
Update configuration system design #295
Conversation
What do you think? @FabianHofmann @fneum @martacki @lisazeyen @koen-vg ? |
I'm trying to get the CI running. Turns out |
I think this looks good! The implementation looks sensible. There seems to still be some default configuration in the renewable:
onwind:
cutout: europe-2013-era5-tutorial
offwind-ac:
cutout: europe-2013-era5-tutorial
offwind-dc:
cutout: europe-2013-era5-tutorial
solar:
cutout: europe-2013-era5-tutorial And on that note, maybe we can also take this opportunity to update atlite:
cutouts:
# Automatically determine geographical bounds and time span.
base:
module: era5 That should be the preferred way nowadays, instead of messing around with coordinates, right? |
YAML does not recognise None, only null, empty string ("") or False.
@FabianHofmann CI is failing due to an
|
yes, i will have a look |
Benching this PR for now. Might be of interest to re-open later. Right now no concrete need or benefit seen. |
Closes #292 .
Changes proposed in this Pull Request
New configuration system:
config.yaml
ofconfig.default.yaml
(instead of fully substituting the configuration file.config/
TODOs
Checklist
Newly introduced dependencies are added toenvs/environment.yaml
andenvs/environment.docs.yaml
.config.default.yaml
,config.tutorial.yaml
, andtest/config.test1.yaml
.Changes in configuration options are also documented indoc/configtables/*.csv
and line references are adjusted indoc/configuration.rst
anddoc/tutorial.rst
.doc/release_notes.rst
is amended in the format of previous release notes.