-
Notifications
You must be signed in to change notification settings - Fork 55
Restructure config #264
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
Restructure config #264
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A great with the split of the sectors! I think it would be good if in the config.yaml each setting would have an explanatory comment and each abbreviation is explained as well. e.g.
tes: true # thermal energy storage (tes)
@lisazeyen I have included explanatory comments and abbreviations. If the proposed config (structure) is fine, I can proceed with updating the config references in the scripts. Before I'll do that (probably time consuming undertaking), we should doublecheck if the main model users agrees with the new structure. |
We'll continue discussions over at PyPSA-Eur once merged. |
This PR restructures the config file to be more structured and comprehensive. It temporarily includes functions to check differences between old and new config files. Closes PyPSA/pypsa-eur#579.
Changelog:
"sector" split into: transport, heating, sector_techs, networks