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

Storage bug fixes #91

Merged
merged 3 commits into from Oct 15, 2023
Merged

Conversation

kingsleynweye
Copy link
Member

Description

Bug fixes in citylearn.energy_model.Storage and how electricity consumption when charging storage devices is calculated.

Issue

NIL

Changes

  • Fixed possibility of citylearn.energy_model.Storage.energy_init less than 0 which, is not physically possible.
  • Fixed case where charged energy is less than what is prescribed by controller causing electricity consumption by storage device to be overestimated.

Screenshots

NIL

Checklist

  • I have tested the changes locally and they work as intended.
  • I have updated the documentation, if applicable.
  • I have added new tests, if applicable.
  • I have added any required dependencies to the requirements.txt file, if applicable.
  • I have followed the project's code style and conventions.

Additional notes

NIL

@kingsleynweye kingsleynweye merged commit f84ff1e into master Oct 15, 2023
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.

None yet

1 participant