Skip to content

Conversation

@byte-bastler
Copy link

The VDI4655 code example in the documentation is not working with the current version of demandlib (v2.2.0). There are two reasons for this:

  1. The house dictionary requires the summer_temperature_limit and winter_temperature_limit entries to be set in order to run the example. When these entries are not set, a KeyError is raised. Even if they become optional again in the future (I saw a change in another commit), it probably doesn't hurt to include them in the code example.

  2. The vdi.Region() constructor no longer accepts a try_region parameter. It seems that climate is used instead.

These changes make the example executable again.

@p-snft
Copy link
Member

p-snft commented Jan 20, 2026

Thank you. Looks like in #50, we only adjusted the example in the examples directory but not the one in the docstring.

Concerning 1: The docstring is meant for the upcoming release, but you are right: adding the parameters does not hurt.

@p-snft p-snft merged commit dfc2475 into oemof:dev Jan 20, 2026
2 checks passed
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

Successfully merging this pull request may close these issues.

2 participants