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

Danfoss Ally: Add support for preheat_status, adaptation variables, regulation setpoint offset #4020

Merged
merged 2 commits into from
Mar 22, 2022

Conversation

AresMaster
Copy link
Contributor

Added support for:

  • regulation setpoint offset
  • adaptation run control
  • adaptation run status
  • adaptation run settings
  • preheat status

obraz

This will allow observation of the valve behaviour and reduce the problem of "random heating" at night.
Koenkk/zigbee2mqtt#7107 (comment)

In case of overheating it also allows a simple change of regulation setpoint offset.

devices/danfoss.js Outdated Show resolved Hide resolved
devices/danfoss.js Outdated Show resolved Hide resolved
@AresMaster
Copy link
Contributor Author

I missed that. I hope everything is all right now.

@AresMaster AresMaster requested a review from Koenkk March 21, 2022 13:12
@Koenkk
Copy link
Owner

Koenkk commented Mar 21, 2022

I think something went wrong while rebasing this pr (e.g. unexpected changes: https://github.com/Koenkk/zigbee-herdsman-converters/pull/4020/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519R3), could you fix that?

@AresMaster AresMaster marked this pull request as ready for review March 22, 2022 07:54
@AresMaster
Copy link
Contributor Author

It should be fine now

@Koenkk
Copy link
Owner

Koenkk commented Mar 22, 2022

perfect!

@Koenkk Koenkk merged commit ce99463 into Koenkk:master Mar 22, 2022
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

2 participants