-
Notifications
You must be signed in to change notification settings - Fork 16
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
Thermo schedule #80
base: master
Are you sure you want to change the base?
Thermo schedule #80
Conversation
698e022
to
d1fd894
Compare
Great progress....now all I need would be the Scheduler for the Aqua :-) |
Yes. |
break; | ||
} | ||
case 0x12: { // Temp Offset | ||
this.tempOfs = buf.readInt8(ofs) / 10; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe an event could be emitted here, to make use of the temperature offset in external plugin code.
For now, it's just a fake setting.
9e0d00a
to
6846c36
Compare
6846c36
to
28149bd
Compare
Are you ok with this PR? I will try to find some time to check and merge it in the next weeks |
After some days of reverse-engineering i actually made it work and did an implementation for it right away.
You can enable this feature as follows:
or
More details of my findings are documented in Ticket #40