башфикс сервисов, ускорение загрузки

custom_components/mega/__init__.py

@@ -1,13 +1,11 @@
 """The mega integration."""
 import asyncio
 import logging
-import typing
 from functools import partial
 
 import voluptuous as vol
-from homeassistant.const import CONF_HOST, CONF_PASSWORD, CONF_PLATFORM, CONF_SCAN_INTERVAL, CONF_ID
-from homeassistant.core import HomeAssistant
-from homeassistant.helpers.entity_component import EntityComponent
+from homeassistant.const import CONF_HOST, CONF_PASSWORD, CONF_SCAN_INTERVAL, CONF_ID
+from homeassistant.core import HomeAssistant, ServiceCall
 from homeassistant.helpers.service import bind_hass
 from homeassistant.components import mqtt
 from homeassistant.config_entries import ConfigEntry
@@ -52,18 +50,18 @@ async def async_setup(hass: HomeAssistant, config: dict):
     conf = config.get(DOMAIN)
     hass.data[DOMAIN] = {}
     hass.services.async_register(
-        DOMAIN, 'save', _save_service, schema=vol.Schema({
+        DOMAIN, 'save', partial(_save_service, hass), schema=vol.Schema({
             vol.Optional('mega_id'): str
         })
     )
     hass.services.async_register(
-        DOMAIN, 'get_port', _get_port, schema=vol.Schema({
+        DOMAIN, 'get_port', partial(_get_port, hass), schema=vol.Schema({
             vol.Optional('mega_id'): str,
             vol.Optional('port'): int,
         })
     )
     hass.services.async_register(
-        DOMAIN, 'run_cmd', _run_cmd, schema=vol.Schema({
+        DOMAIN, 'run_cmd', partial(_run_cmd, hass), schema=vol.Schema({
             vol.Required('port'): int,
             vol.Required('cmd'): str,
             vol.Optional('mega_id'): str,
@@ -74,9 +72,10 @@ async def async_setup(hass: HomeAssistant, config: dict):
     if CONF_HOST in conf:
         conf = {DEF_ID: conf}
     for id, data in conf.items():
+        _LOGGER.warning('YAML configuration is deprecated, please use web-interface')
         await _add_mega(hass, id, data)
 
-    for id, hub in hass.data[DOMAIN].__items__():
+    for id, hub in hass.data[DOMAIN].items():
         _POLL_TASKS[id] = asyncio.create_task(hub.poll())
     return True
 
@@ -133,8 +132,8 @@ async def async_remove_entry(hass, entry) -> None:
     unsub()
 
 
-@bind_hass
-async def _save_service(hass: HomeAssistant, mega_id=None):
+async def _save_service(hass: HomeAssistant, call: ServiceCall):
+    mega_id = call.data.get('mega_id')
     if mega_id:
         hub: MegaD = hass.data[DOMAIN][mega_id]
         await hub.save()
@@ -144,7 +143,9 @@ async def _save_service(hass: HomeAssistant, mega_id=None):
 
 
 @bind_hass
-async def _get_port(hass: HomeAssistant, port=None, mega_id=None):
+async def _get_port(hass: HomeAssistant, call: ServiceCall):
+    port = call.data.get('port')
+    mega_id = call.data.get('mega_id')
     if mega_id:
         hub: MegaD = hass.data[DOMAIN][mega_id]
         if port is None:
@@ -158,8 +159,12 @@ async def _get_port(hass: HomeAssistant, port=None, mega_id=None):
             else:
                 await hub.get_port(port)
 
+
 @bind_hass
-async def _run_cmd(hass: HomeAssistant, port: int, cmd: str, mega_id=None):
+async def _run_cmd(hass: HomeAssistant, call: ServiceCall):
+    port = call.data.get('port')
+    mega_id = call.data.get('mega_id')
+    cmd = call.data.get('cmd')
     if mega_id:
         hub: MegaD = hass.data[DOMAIN][mega_id]
         await hub.send_command(port=port, cmd=cmd)

custom_components/mega/binary_sensor.py

@@ -1,6 +1,5 @@
 """Platform for light integration."""
 import asyncio
-import json
 import logging
 
 import voluptuous as vol
@@ -18,7 +17,6 @@ from homeassistant.const import (
     CONF_ID
 )
 from homeassistant.core import HomeAssistant
-from homeassistant.helpers.restore_state import RestoreEntity
 from .entities import BaseMegaEntity
 
 from .hub import MegaD
@@ -63,13 +61,16 @@ async def async_setup_entry(hass: HomeAssistant, config_entry: ConfigEntry, asyn
     mid = config_entry.data[CONF_ID]
     hub: MegaD = hass.data['mega'][mid]
     devices = []
-    async for port, pty, m in hub.scan_ports():
-        if pty == "0":
-            sensor = MegaBinarySensor(mega_id=mid, port=port)
-            devices.append(sensor)
 
-    async_add_devices(devices)
+    async def scan():
+        async for port, pty, m in hub.scan_ports():
+            if pty == "0":
+                sensor = MegaBinarySensor(mega_id=mid, port=port)
+                devices.append(sensor)
 
+        async_add_devices(devices)
+
+    asyncio.create_task(scan())
 
 class MegaBinarySensor(BinarySensorEntity, BaseMegaEntity):
 

custom_components/mega/config_flow.py

@@ -9,7 +9,7 @@ from homeassistant.components import mqtt
 from homeassistant.config_entries import ConfigEntry
 from homeassistant.const import CONF_HOST, CONF_ID, CONF_PASSWORD, CONF_SCAN_INTERVAL
 from homeassistant.core import callback
-from .const import DOMAIN, CONF_PORT_TO_SCAN, CONF_RELOAD, CONF_INVERT  # pylint:disable=unused-import
+from .const import DOMAIN, CONF_PORT_TO_SCAN, CONF_RELOAD  # pylint:disable=unused-import
 from .hub import MegaD
 from . import exceptions
 

custom_components/mega/hub.py

@@ -140,36 +140,45 @@ class MegaD:
     async def save(self):
         await self.send_command(cmd='s')
 
-    async def get_port(self, port, get_value=False):
-        if get_value:
-            ftr = asyncio.get_event_loop().create_future()
+    async def get_port(self, port):
+        """
+        Опрашивает порт с помощью mqtt. Ждет ответ, возвращает ответ.
 
-            def cb(msg):
-                try:
-                    ftr.set_result(json.loads(msg.payload).get('value'))
-                except Exception as exc:
-                    self.lg.warning(f'could not parse {msg.payload}: {exc}')
-                    ftr.set_result(None)
-            unsub = await self.mqtt.async_subscribe(
-                topic=f'{self.mqtt_id}/{port}',
-                msg_callback=cb,
-                qos=1,
-            )
+        :param port:
+        :return:
+        """
+        ftr = asyncio.get_event_loop().create_future()
 
-        self.lg.debug(
-            f'get port: %s', port
-        )
-        async with self.lck:
-            await self.mqtt.async_publish(
-                topic=f'{self.mqtt_id}/cmd',
-                payload=f'get:{port}',
-                qos=0,
-                retain=False,
-            )
-            await asyncio.sleep(0.1)
+        def cb(msg):
 
-        if get_value:
             try:
+                if '"value":NA' in msg.payload.decode():
+                    ftr.set_result(None)
+                    return
+                ret = json.loads(msg.payload).get('value')
+                ftr.set_result(ret)
+            except Exception as exc:
+                ret = None
+                self.lg.exception(f'while parsing response from port {port}: {msg.payload}')
+                ftr.set_result(None)
+
+            self.lg.debug(
+                f'port: %s response: %s', port, ret
+            )
+
+        async with self.lck:
+            try:
+                unsub = await self.mqtt.async_subscribe(
+                    topic=f'{self.mqtt_id}/{port}',
+                    msg_callback=cb,
+                    qos=1,
+                )
+                await self.mqtt.async_publish(
+                    topic=f'{self.mqtt_id}/cmd',
+                    payload=f'get:{port}',
+                    qos=1,
+                    retain=False,
+                )
                 return await asyncio.wait_for(ftr, timeout=2)
             except asyncio.TimeoutError:
                 self.lg.warning(f'timeout on port {port}')
@@ -259,7 +268,8 @@ class MegaD:
             return pty, m
 
     async def scan_ports(self,):
-        for x in range(37):
-            ret = await self.scan_port(x)
-            if ret:
-                yield [x, *ret]
+        async with self.lck:
+            for x in range(38):
+                ret = await self.scan_port(x)
+                if ret:
+                    yield [x, *ret]

custom_components/mega/light.py

@@ -1,6 +1,5 @@
 """Platform for light integration."""
 import asyncio
-import json
 import logging
 
 import voluptuous as vol
@@ -19,7 +18,6 @@ from homeassistant.const import (
     CONF_ID
 )
 from homeassistant.core import HomeAssistant
-from homeassistant.helpers.restore_state import RestoreEntity
 from .entities import BaseMegaEntity
 
 from .hub import MegaD
@@ -80,11 +78,15 @@ async def async_setup_entry(hass: HomeAssistant, config_entry: ConfigEntry, asyn
     mid = config_entry.data[CONF_ID]
     hub: MegaD = hass.data['mega'][mid]
     devices = []
-    async for port, pty, m in hub.scan_ports():
-        if pty == "1" and m in ['0', '1']:
-            light = MegaLight(mega_id=mid, port=port, dimmer=m == '1')
-            devices.append(light)
-    async_add_devices(devices)
+
+    async def scan_ports():
+        async for port, pty, m in hub.scan_ports():
+            if pty == "1" and m in ['0', '1']:
+                light = MegaLight(mega_id=mid, port=port, dimmer=m == '1')
+                devices.append(light)
+        async_add_devices(devices)
+
+    asyncio.create_task(scan_ports())
 
 
 class MegaLight(LightEntity, BaseMegaEntity):
@@ -119,7 +121,7 @@ class MegaLight(LightEntity, BaseMegaEntity):
         return self._state == 'ON'
 
     async def async_turn_on(self, brightness=None, **kwargs) -> None:
-        brightness = brightness or self.brightness
+        brightness = brightness or self.brightness or 255
         if self.dimmer and brightness == 0:
             cmd = 255
         elif self.dimmer:

custom_components/mega/sensor.py

@@ -1,7 +1,6 @@
 """Platform for light integration."""
+import asyncio
 import logging
-
-import typing
 import voluptuous as vol
 
 from homeassistant.components.sensor import (
@@ -87,30 +86,33 @@ async def async_setup_entry(hass: HomeAssistant, config_entry: ConfigEntry, asyn
     mid = config_entry.data[CONF_ID]
     hub: MegaD = hass.data['mega'][mid]
     devices = []
-    async for port, pty, m in hub.scan_ports():
-        if pty == "3":
-            values = await hub.get_port(port, get_value=True)
-            lg.debug(f'values: %s', values)
-            if values is None:
-                continue
-            if isinstance(values, str) and TEMP_PATT.search(values):
-                    values = {TEMP: values}
-            elif not isinstance(values, dict):
-                values = {None: values}
-            for key in values:
-                hub.lg.debug(f'add sensor {W1}:{key}')
-                sensor = _make_entity(
-                    mid=mid,
-                    port=port,
-                    conf={
-                        CONF_TYPE: W1,
-                        CONF_KEY: key,
-                    })
-                devices.append(sensor)
-                hub.sensors.append(sensor)
 
-    async_add_devices(devices)
+    async def scan():
+        async for port, pty, m in hub.scan_ports():
+            if pty == "3":
+                values = await hub.get_port(port)
+                lg.debug(f'values: %s', values)
+                if values is None:
+                    continue
+                if isinstance(values, str) and TEMP_PATT.search(values):
+                        values = {TEMP: values}
+                elif not isinstance(values, dict):
+                    values = {None: values}
+                for key in values:
+                    hub.lg.debug(f'add sensor {W1}:{key}')
+                    sensor = _make_entity(
+                        mid=mid,
+                        port=port,
+                        conf={
+                            CONF_TYPE: W1,
+                            CONF_KEY: key,
+                        })
+                    devices.append(sensor)
+                    hub.sensors.append(sensor)
 
+        async_add_devices(devices)
+
+    asyncio.create_task(scan())
 
 class Mega1WSensor(BaseMegaEntity):
 

custom_components/mega/switch.py

@@ -1,5 +1,4 @@
 """Platform for light integration."""
-import json
 import logging
 
 import voluptuous as vol
@@ -13,11 +12,7 @@ from homeassistant.const import (
     CONF_PLATFORM,
     CONF_PORT,
 )
-from homeassistant.core import HomeAssistant
-from homeassistant.helpers.restore_state import RestoreEntity
 from .entities import BaseMegaEntity
-
-from .hub import MegaD
 from .const import CONF_DIMMER, CONF_SWITCH
 
 _LOGGER = logging.getLogger(__name__)

readme.md

@@ -3,15 +3,17 @@
 Интеграция с [MegaD-2561](https://www.ab-log.ru/smart-house/ethernet/megad-2561)
 
 ## Основные особенности:
-- Настройка как из yaml так и из веб-интерфейса
-- При настройки из веба все порты автоматически добавляются как устройства (для обычных релейных выходов создается 
+- Все порты автоматически добавляются как устройства (для обычных релейных выходов создается 
   `light`, для шим - `light` с поддержкой яркости, для цифровых входов `binary_sensor`, для температурных датчиков
   `sensor`)
 - Возможность работы с несколькими megad
 - Обратная связь по mqtt
 - Команды выполняются друг за другом без конкурентного доступа к ресурсам megad
+- Поддержка температурных датчиков в режиме шины
+
 ## Устройства
 Поддерживаются устройства: light, switch, binary_sensor, sensor. light может работать как диммер
+
 ## Установка
 Рекомендованнй способ - через [HACS](https://hacs.xyz/docs/installation/installation).
 После установки HACS, нужно перейти в меню hacs -> integrations, далее в верхнем правом углу
@@ -25,61 +27,13 @@ wget -q -O - https://raw.githubusercontent.com/andvikt/mega_hacs/master/install.
 Перезагрузить HA
 
 Для обновления повторить
+
 ## Зависимости
 Перед использованием необходимо настроить интеграцию mqtt в HomeAssistant
 
 ## Настройка из веб-интерфейса
 `Настройки` -> `Интеграции` -> `Добавить интеграцию` в поиске ищем mega
 
-## Пример настройки с помощью yaml:
-```yaml
-mega: 
-  mega1:
-    host: 192.168.0.14
-    name: hello
-    password: sec
-    mqtt_id: mega # это id в конфиге меги
-
-light:
-  - platform: mega
-    mega1:
-      switch:
-        - 1 # можно просто перечислить порты
-        - 2
-        - 3
-      dimmer:
-        - port: 7
-          name: hello # можно использовать расширенный вариант с названиями
-        - 9
-        - 10
-
-binary_sensor:
-  - platform: mega
-    mega1:
-      - port: 16
-        name: sensor1
-      - port: 18
-        name: sensor2
-
-sensor:
-  - platform: mega
-    mega1:
-      - port: 10
-        name: some temp
-        type: w1
-        key: temp
-      - port: 10
-        name: some hum
-        type: w1
-        key: hum
-
-switch:
-  - platform: mega
-    mega1:
-      - 11
-
-```
-
 ## Сервисы
 ```yaml
 save:
@@ -94,7 +48,7 @@ get_port:
   description: Запросить текущий статус порта (или всех)
   fields:
     mega_id:
-      description: ID меги, можно оставить пустым, тогда будут сохранены все зарегистрированные меги
+      description: ID меги, можно оставить пустым, тогда будут порты всех зарегистрированных мег
       example: "mega"
     port:
       description: Номер порта (если не заполнять, будут запрошены все порты сразу)
@@ -105,7 +59,7 @@ run_cmd:
   fields:
 
     mega_id:
-      description: ID меги, можно оставить пустым, тогда будут сохранены все зарегистрированные меги
+      description: ID меги
       example: "mega"
     port:
       description: Номер порта (это не порт, которым мы управляем, а порт с которого шлем команду)
@@ -116,10 +70,6 @@ run_cmd:
 
 ```
 
-## Состояния
-Так же каждое устройство megad опрашивается на предмет работоспособности, текущий статус
-хранится в mega.<id>
-
 ## Отладка
 Если возникают проблемы, можно включить детальный лог, для этого в конфиг добавить:
 ```yaml
@@ -127,4 +77,4 @@ logger:
   default: info
   logs:
     custom_components.mega: debug
-```
+```