Conversation
|
I agree with removing the full copy to defaults to config.yaml. this lead to a lot of issues. however removing the |
|
So you would remove the complete default copy, but keep Sounds good to me. You would still have the fourfold chain if you pass your own config file, and it is not super obvious that a |
|
@lkstrp Is there any update on this? |
|
No, we just need to choose one of the two options:
I guess people would prefer the second |
I personally find that having I think we should remove the copy behaviour as @FabianHofmann already suggested, but we could keep the tl;dr I agree with the upside of option two if the copy behaviour is removed |
This sounds like the best compromise. The important aspects to me are:
|
|
This is good to go now |
Closes # (if applicable).
Changes proposed in this Pull Request
config/config.yamlfrom the snakemake config chainconfig/config.default.yaml,config/plotting.default.yaml,config/config.yamland then your passed config. For PyPSA-DE another conifg is addedcopy_default_filesis very confusing and not good snakemake practice I would assume. The createdconfig.yamlcan lead to an artifact being used without the user realising itconfig.template.yamlbut not run it by defaultRemoving this is quite breaking. If we agree, I'll adjust the docs, release notes etc.
Checklist
envs/environment.yaml.config/config.default.yaml.doc/configtables/*.csv.doc/data_sources.rst.doc/release_notes.rstis added.