-
Notifications
You must be signed in to change notification settings - Fork 4
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
Fix/heat assets costs #189
Conversation
unfortunately these changes stand in conflict with the new file structur (#190 ), where the PARAMETERS_mvs_input file will be deleted. Can the changes be applied to the new file structure directly? |
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.
das einzige ist, dass es vielleicht sinnvoll wäre sich eine einheitliche Zitierweise im Read the docs zu überlegen. Aber das kann auch später für alles zusammen noch geändert werden..
That's a good point! This will have to be done solving issue #154 |
With this PR the cost parameters of the heat assets are adapted since the parameter
specific_costs_om
of the heat pump was wrong. Further the parameters of an air to water heat pump are added to the template in energyConsumption.csvThe following steps were realized, as well (if applies):
black . --exclude docs/
)Please mark above checkboxes as following:
❌ Check not applicable to this PR
For more information on how to contribute check the CONTRIBUTING.md.