-
-
Notifications
You must be signed in to change notification settings - Fork 31.3k
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
Support for the Quirky Nimbus #16520
Conversation
max_position = attributes["max_position"] | ||
|
||
self.wink.set_configuration(min_value, max_value, rotation, scale=scale, ticks=ticks, | ||
min_position=min_position, max_position=max_position) |
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.
line too long (89 > 79 characters)
min_position = attributes["min_position"] | ||
max_position = attributes["max_position"] | ||
|
||
self.wink.set_configuration(min_value, max_value, rotation, scale=scale, ticks=ticks, |
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.
line too long (93 > 79 characters)
} | ||
|
||
def set_configuration(self, **kwargs): | ||
"""Set the dial config, anything not sent will default to current setting.""" |
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.
line too long (85 > 79 characters)
if service.service == SERVICE_SET_DIAL_CONFIG: | ||
_dial.set_configuration(**service.data) | ||
if service.service == SERVICE_SET_DIAL_STATE: | ||
_dial.wink.set_state(service.data.get("value"), service.data.get("labels")) |
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.
line too long (95 > 79 characters)
5e91bc3
to
87563bf
Compare
Description:
Wink has a legacy device called the Nimbus which is a four dial digital display capable of show values on a gauge and a display.
Each dial of the nimbus will be displayed as it's own entity with a state of the current dials "value" and attributes containing the dials configuration as well as current state.
These dials can be updated based on another value in Home Assistant to provide type of Home Assistant dashboard!
Two new wink services will be added if the users account has a nimbus associated with it.
@siam28 requested this be added on the python-wink github. @siam28 if you could test this that would be awesome.
Related issue (if applicable): fixes climate issue reported in the forums where some attributes weren't showing up.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.io#6246
Checklist:
tox
. Your PR cannot be merged unless tests passIf user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
requirements_all.txt
by runningscript/gen_requirements_all.py
.