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

Thermo schedule #80

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Thermo schedule #80

wants to merge 4 commits into from

Conversation

tobekas
Copy link

@tobekas tobekas commented Nov 27, 2019

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:

// in your module.exports:
Schedule = require('fakegato-history').Schedule(homebridge);
...
// in your code:
let thermoScheduler = new Schedule('thermo', log);
thermoScheduler.registerScheduleEvents(thermostatService);

or

// in your includes:
var fakegatoHistory = require('fakegato-history');
var Schedule;
...
// in your module.exports:
Schedule = fakegatoHistory.Schedule(homebridge);
...
// in your code:
let thermoScheduler = new Schedule('thermo', log);
thermoScheduler.registerScheduleEvents(thermostatService);

More details of my findings are documented in Ticket #40

@ctschach
Copy link

Great progress....now all I need would be the Scheduler for the Aqua :-)

@tobekas
Copy link
Author

tobekas commented Nov 27, 2019

Yes.
The schedule for Aqua (as well as for Eve Light Switch, Eve Energy and Eve Energy Strip) is working with TLV-encoded custom characteristics E863F11D (setConfiguration) and E863F131 (getConfiguration). This is different from the Thermo commands and seems to be more complicated, because there is much more functionality. But I've also found out some things about it.

@tobekas tobekas mentioned this pull request Nov 27, 2019
break;
}
case 0x12: { // Temp Offset
this.tempOfs = buf.readInt8(ofs) / 10;
Copy link
Author

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.

@simont77
Copy link
Owner

simont77 commented Dec 4, 2019

Are you ok with this PR? I will try to find some time to check and merge it in the next weeks

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.

3 participants