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

Feature: select active temperature sensor #587

Closed
wants to merge 1 commit into from

Conversation

etiennedub
Copy link

close #470

New feature: Add selection for the active temperature sensor.

It works by adding new Service.Switches, for the main thermostat and for each temperature sensor. When pressed the selected sensor is set to active and others are disable. IMO the behavior is a bit weird, because we don't use the switch as intended (it's more like a button press with state for visualization).

I was looking for some drop-down menu or radio button-like service but I haven't found any. If you have any better idea fell free to comment or commit.

@adriancable
Copy link
Collaborator

This only works for thermostats on the REST API (which is only about 30% of total thermostats). Since thermostat assignment to the REST vs protobuf API is random, your PR will only work for 30% of people (without people being able to know in advance whether they are in the 30% or 70%). So I cannot accept it in this form.

@etiennedub
Copy link
Author

I understand, to be honest I could not say that I really understand the whole API. I mostly mimic what I have on my network.

Is there a way to "switch" the API selection? Does it depends on devices or it's really random?

@adriancable
Copy link
Collaborator

@etiennedub - there's no way to influence the API used for devices. For the 3rd gen Learning Thermostat, it appears to be random. (Probably internally it's selected based on serial number or something similar, but I don't have any insight into how that occurs.)

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.

Add support for changing the active temperature sensor
2 participants