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

Fix activation of Hp when not under control of an EM #923

Conversation

danielfeismann
Copy link
Member

resolves #922

@danielfeismann danielfeismann self-assigned this Aug 24, 2024
@danielfeismann danielfeismann added the bug Something isn't working label Aug 24, 2024
@danielfeismann
Copy link
Member Author

Description what caused the error / wrong behaviour

When the lower temperature limit of the heat house is reached, the heat pump starts to operate, but not immediately when the limit is reached. This is caused by the fact that the maybeThreshold is calculated correctly, but is not added to the additionalActivationTicks.

Copy link
Member

@sebastian-peter sebastian-peter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good! Found only a tiny way to improve the PR.

Co-authored-by: Sebastian Peter <[email protected]>
@sebastian-peter sebastian-peter merged commit 67ad290 into dev Aug 29, 2024
4 checks passed
@sebastian-peter sebastian-peter deleted the df/#922-Fix-activation-of-Hp-when-not-under-control-of-an-EM branch August 29, 2024 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix activation of Hp when not under control of an EM
2 participants