5dd45efac3
* Updated ISY component to not overwrite state_attributes. The ISY component included an ISYDevice base class that is used by all of the isy994 platforms. This still overwrote the state_attributes property instead of the more appropriate device_state_attributes property. This was also repeated in the isy994 light platform. Both of these were addressed. This also fixes issue #5428. * Removed custom state attributes from ISY lights. The brightness attribute need not be manually reported by the isy994 light platform. * Removed ISY Node cleanup. The ISY entities don’t really need to unsubscribe themselves while hass is shutting down. Because these updates are not sent in a thread, there is no negative impact from shutting down without unsubscribing. This greatly speeds up hass shutdown. * Removed unused attribute from isy994 light platform. * Cleaned up ISY994 light entity class. 1) Removed the state property. This property is set in the Entity base class and shouldn’t be overridden here. 2) Set the brightness property. This is the proper way of setting the brightness for the Light base class. 3) Removed properties that are now unused because of these changes.
70 lines
2.1 KiB
Python
70 lines
2.1 KiB
Python
"""
|
|
Support for ISY994 lights.
|
|
|
|
For more details about this platform, please refer to the documentation at
|
|
https://home-assistant.io/components/light.isy994/
|
|
"""
|
|
import logging
|
|
from typing import Callable
|
|
|
|
from homeassistant.components.light import (
|
|
Light, SUPPORT_BRIGHTNESS)
|
|
import homeassistant.components.isy994 as isy
|
|
from homeassistant.const import STATE_ON, STATE_OFF
|
|
from homeassistant.helpers.typing import ConfigType
|
|
|
|
_LOGGER = logging.getLogger(__name__)
|
|
|
|
UOM = ['2', '51', '78']
|
|
STATES = [STATE_OFF, STATE_ON, 'true', 'false', '%']
|
|
|
|
|
|
# pylint: disable=unused-argument
|
|
def setup_platform(hass, config: ConfigType,
|
|
add_devices: Callable[[list], None], discovery_info=None):
|
|
"""Set up the ISY994 light platform."""
|
|
if isy.ISY is None or not isy.ISY.connected:
|
|
_LOGGER.error('A connection has not been made to the ISY controller.')
|
|
return False
|
|
|
|
devices = []
|
|
|
|
for node in isy.filter_nodes(isy.NODES, units=UOM,
|
|
states=STATES):
|
|
if node.dimmable or '51' in node.uom:
|
|
devices.append(ISYLightDevice(node))
|
|
|
|
add_devices(devices)
|
|
|
|
|
|
class ISYLightDevice(isy.ISYDevice, Light):
|
|
"""Representation of an ISY994 light devie."""
|
|
|
|
def __init__(self, node: object) -> None:
|
|
"""Initialize the ISY994 light device."""
|
|
isy.ISYDevice.__init__(self, node)
|
|
|
|
@property
|
|
def is_on(self) -> bool:
|
|
"""Get whether the ISY994 light is on."""
|
|
return self.value > 0
|
|
|
|
@property
|
|
def brightness(self) -> float:
|
|
"""Get the brightness of the ISY994 light."""
|
|
return self.value
|
|
|
|
def turn_off(self, **kwargs) -> None:
|
|
"""Send the turn off command to the ISY994 light device."""
|
|
if not self._node.off():
|
|
_LOGGER.debug('Unable to turn on light.')
|
|
|
|
def turn_on(self, brightness=None, **kwargs) -> None:
|
|
"""Send the turn on command to the ISY994 light device."""
|
|
if not self._node.on(val=brightness):
|
|
_LOGGER.debug('Unable to turn on light.')
|
|
|
|
@property
|
|
def supported_features(self):
|
|
"""Flag supported features."""
|
|
return SUPPORT_BRIGHTNESS
|