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

BUG: configuration file example is broken #78

Closed
ahobeost opened this issue Jun 25, 2024 · 2 comments
Closed

BUG: configuration file example is broken #78

ahobeost opened this issue Jun 25, 2024 · 2 comments
Assignees

Comments

@ahobeost
Copy link
Contributor

The example for the configuration file is no longer complete.
Some changes to the EnergyNetwork have broken the example.

I think mergeBuses is a required argument to setFromExcel.
When it stays None, the following error is raised:
image

https://github.com/SPF-OST/optihood/blob/main/optihood/buildings.py line 108.

@nehadimri1991
Copy link
Collaborator

nehadimri1991 commented Jun 27, 2024

mergebuses is a required argument if you set mergeLinkBuses to True. It's a list specifying the names of buses to be merged. In this case you would need to set it to ["electricity", "space_heat", "domestic_hot_water"]. This would be changed in my next updates to optihood, the idea is to have only mergebuses and get rid of the mergeLinkBuses boolean parameter.

@ahobeost
Copy link
Contributor Author

ahobeost commented Jul 4, 2024

Thanks!
Fixed on the current branch, will close once merged with main.

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

No branches or pull requests

2 participants