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.

Sign up for GitHub

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

Jump to bottom

Homekit thermostat group not working #1053

Closed
rrgeorge opened this issue Jun 22, 2016 · 3 comments
Closed

Homekit thermostat group not working #1053

rrgeorge opened this issue Jun 22, 2016 · 3 comments

Comments

@rrgeorge
Copy link

After updating to the latest snapshot and updating the tags, the thermostat group for HomeKit no longer works. The only item I'm seeing on iOS is the CurrentTemperature item as a temperature sensor, not the thermostat group as a thermostat. The following is my relevant items:

Group main_thermostat "Main Thermostat" (ecobee) [ "Thermostat" ]
Number actualTemperature "Current Temperature [%.1f °F]" <temperature> (main_thermostat) [ "CurrentTemperature" ] { ecobee="<[316561513825#runtime.actualTemperature]" }
Number desiredTemp "Target Temperature [%.1 °F]" <temperature> (main_thermostat) [ "TargetTemperature" ]
String hvacMode "Mode [%s]"            <none> (main_thermostat) [ "homekit:HeatingCoolingMode" ] { ecobee="=[316561513825#settings.hvacMode]" }
@kaikreuzer
Copy link
Member

@beowulfe Can you help?

@andylintner
Copy link

On it

andylintner added a commit to andylintner/openhab2 that referenced this issue Jun 23, 2016
@andylintner
Allow CurrentTemperature on HomeKit Thermostat
898b7bd 
In the initial implementation, characteristics and accessories had their own unique naming convention. With the ESH ontology, CurrentTemperature became both an individual accessory and a characteristic of a Thermostat.

With this change, we look at the groups an item belongs to before deciding if it is an accessory or a characteristic - if a group is tagged with a recognized value, we treat the item as a characteristic.

Fixes openhab#1053

Signed-off-by: Andy Lintner <[email protected]>
@andylintner andylintner mentioned this issue Jun 23, 2016
Merged
kaikreuzer pushed a commit that referenced this issue Jul 4, 2016
@andylintner @kaikreuzer
Allow CurrentTemperature on HomeKit Thermostat (#1056)
719b079 
In the initial implementation, characteristics and accessories had their own unique naming convention. With the ESH ontology, CurrentTemperature became both an individual accessory and a characteristic of a Thermostat.

With this change, we look at the groups an item belongs to before deciding if it is an accessory or a characteristic - if a group is tagged with a recognized value, we treat the item as a characteristic.

Fixes #1053

Signed-off-by: Andy Lintner <[email protected]>
@kaikreuzer
Copy link
Member

Thanks, sorry for the late merge, I missed the PR!

doubled-ca pushed a commit to doubled-ca/openhab-addons that referenced this issue Aug 14, 2016
@andylintner @doubled-ca
Allow CurrentTemperature on HomeKit Thermostat (openhab#1056)
ad03455 
In the initial implementation, characteristics and accessories had their own unique naming convention. With the ESH ontology, CurrentTemperature became both an individual accessory and a characteristic of a Thermostat.

With this change, we look at the groups an item belongs to before deciding if it is an accessory or a characteristic - if a group is tagged with a recognized value, we treat the item as a characteristic.

Fixes openhab#1053

Signed-off-by: Andy Lintner <[email protected]>
@wborn wborn added the homekit label Dec 16, 2018
Flole998 pushed a commit to Flole998/openhab-addons that referenced this issue Dec 30, 2021
@wborn @cweitkamp
Inject SchedulerImpl into DelegatedSchedulerImpl to fix waiting bundl…
c9232d9 
…es (openhab#1054)

Fixes openhab#1053

Signed-off-by: Wouter Born <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@andylintner @kaikreuzer @rrgeorge @wborn