PyPlumIO 0.4.0__py3-none-any.whl → 0.4.1__py3-none-any.whl

{PyPlumIO-0.4.0.dist-info → PyPlumIO-0.4.1.dist-info}/METADATA

@@ -1,10 +1,10 @@
 Metadata-Version: 2.1
 Name: PyPlumIO
-Version: 0.4.0
+Version: 0.4.1
 Summary: PyPlumIO is a native ecoNET library for Plum ecoMAX controllers.
 Author-email: Denis Paavilainen <denpa@denpa.pro>
 License: MIT License
-Project-URL: Documentation, https://pypulmio.denpa.pro
+Project-URL: Documentation, https://pyplumio.denpa.pro
 Project-URL: Source Code, https://github.com/denpamusic/PyPlumIO
 Project-URL: Bug Tracker, https://github.com/denpamusic/PyPlumIO/issues
 Keywords: home,automation,heating

{PyPlumIO-0.4.0.dist-info → PyPlumIO-0.4.1.dist-info}/RECORD

@@ -1,15 +1,15 @@
 pyplumio/__init__.py,sha256=HZuhLA98rflVEU5cc2-UpboDSJUW1e6Eocfdok9MHZM,1273
 pyplumio/__main__.py,sha256=j8fh3P6yGwB1qumdUlW8X22-DOwisdo4IcUEmlYPGbM,497
-pyplumio/_version.py,sha256=QiCQVGOicQZY3DBr4tYPU9egzT1VgzOJGfLt9Q26uyU,160
+pyplumio/_version.py,sha256=0_dB-SwRXzCHowA3kD_9MmecjmUqQiIbDwJmtaXjluI,160
 pyplumio/connection.py,sha256=ABeJFbnqTq89i5J1TRR2sdQeRrq1Pewdi_TaDjzSOOI,5494
-pyplumio/const.py,sha256=WfV68bd99IWsKHQ2bWW9Rl8xHvgfWrt8cyb2kjs5od4,2965
+pyplumio/const.py,sha256=XGuoUVPnKSeguNk01mtTXNbAzUBQITRaoAumBoaIyD4,3007
 pyplumio/exceptions.py,sha256=cpLGRl9db6cU1Z6WXhQ7yl1V7Te193QymWiOoRF8KAI,798
 pyplumio/filters.py,sha256=X_Q_DE9M_yyw2OWyr_-IargtAWpW9CAI3jIhoIXKhaA,5648
 pyplumio/protocol.py,sha256=h90uhCcqscukJ0GVX6rYv_LH5UsE2om87ZXo3EWZ77s,6223
 pyplumio/stream.py,sha256=n1dae3oKnnU0XWsVgzVszXPdVXkYChdw-bUTKcHr05s,3738
 pyplumio/util.py,sha256=oKKO9eJmrmXf0XHZG5P4fGR_R1jMSU_BAXU82ZnDS8Y,2603
-pyplumio/devices/__init__.py,sha256=R1q2YPhpJk0C-6kEP8sDv5ywxdgGJqLRSCdrhv2MzWQ,4904
-pyplumio/devices/ecomax.py,sha256=bR5An1qTTLeJhSb2Vq07dHYAhH9OpoX2754LThDKOXk,14693
+pyplumio/devices/__init__.py,sha256=DIT3wV0BDU8y1zaM5j9EAhNd5MbTsuCElLZBvZ-3GH8,4871
+pyplumio/devices/ecomax.py,sha256=L74fUjFE3q4lsu4CccsMxCFenljmY0sF1D1CYLoRBDc,15223
 pyplumio/devices/ecoster.py,sha256=IqyyLDqasDSK9taFGlxDoX8Dl8fYOKDL00RB4zX-tZs,314
 pyplumio/devices/mixer.py,sha256=iODTCYHt29XNCam0hnGMdP8VN1uS24b489KD5-9M6-M,1919
 pyplumio/devices/thermostat.py,sha256=J8Iq5jfRmvoFQH2nU-mXNJeGMekp4lqJhtdS2qTVKRM,1701
@@ -30,14 +30,14 @@ pyplumio/helpers/uid.py,sha256=Gt1l9gs-RMummmpfu5IoRokGurxRzJeWZrfwiyeGR9c,1044
 pyplumio/structures/__init__.py,sha256=imqgigigBkwDpHKLwiLTrAh0vqaYcvnm8Y5676uT07Y,1482
 pyplumio/structures/alerts.py,sha256=_AIsA_Uvp2xm8JiQ7a3jt7VuZWcFiNHYtZTOHBAFIkQ,2644
 pyplumio/structures/data_schema.py,sha256=wOWEtoqjtn2qq-Gxb-HParxTnmAjsOjIBxp211Qn66I,1174
-pyplumio/structures/ecomax_parameters.py,sha256=slqV3vntCb6SvKnHb47azuUlMbChD-E1owNzvqNXpxc,15870
+pyplumio/structures/ecomax_parameters.py,sha256=8y1ZFN9NKoiKt_zxW-_m6QBsWssQ2A0uK6oQv6E4LcA,16082
 pyplumio/structures/fan_power.py,sha256=nlurG1-sPyUQbasuxZChwDW4ml3Mp_Jc5RkUpv5NDzg,859
 pyplumio/structures/frame_versions.py,sha256=LqqlC7CVyRDfBMw1Y2Tt0ZaDdlj1K3nlWbb3C3LwzD4,1183
 pyplumio/structures/fuel_consumption.py,sha256=LxQf0Vg7-XnikD0otXvWep6nIP8mK3j_P4cB2xjWwSk,961
 pyplumio/structures/fuel_level.py,sha256=_u6WMLpnIl0W3jA1QKYG708s5PdaaIM35yeEZZ8Srik,800
 pyplumio/structures/lambda_sensor.py,sha256=gIsonwMOtKXhp_v55yKxUm1XNeUGvRkHUI0JwDSOAsY,1315
 pyplumio/structures/load.py,sha256=wltZnHmEE4yk9DzYpwPRs4a8Z8YS7IhBqG81Kpe5c-s,772
-pyplumio/structures/mixer_parameters.py,sha256=W2cG3QpJ1lp3z_LYfx1LBPq2puGR6sWBFMJcjRhrfxo,6417
+pyplumio/structures/mixer_parameters.py,sha256=gShc2ST0IQcfHFhmKVgkz8s1DUdqgVBwIZl_PSXJry4,6115
 pyplumio/structures/mixer_sensors.py,sha256=4AEFPlV-seESz4DDNn6kUl5OJWCaXRejEAtSX2_JdvU,1729
 pyplumio/structures/modules.py,sha256=jFO723D-40Ac1rqchDj7ULWARx-nnx_3i9Tjnn_Llxo,2225
 pyplumio/structures/network_info.py,sha256=-Pz9nuAsTA2vcwNaBkgkvsTfavpikctYiUmrMa6EjCc,3469
@@ -51,10 +51,10 @@ pyplumio/structures/regulator_data.py,sha256=GOfnFcoLN-3vhSIg4GLRDh_5uEWZHYk9E5Y
 pyplumio/structures/schedules.py,sha256=YFDHYaiygc60mQm9EzrXXnMic9EKQsi-CwDM1jBECUQ,6135
 pyplumio/structures/statuses.py,sha256=BeSOabFIewiUBxAXgsuxzjRi4Zl-_O2qDbn61w7DtUU,1036
 pyplumio/structures/temperatures.py,sha256=lPRYCCciaGTBU_ShWwz_bWHvsfIRL3n16CnTbZK9iC4,2298
-pyplumio/structures/thermostat_parameters.py,sha256=zQERSx_Ut6V3oFtl_dDxqyDpNu02LhrBhMrgqPHLG_c,6664
+pyplumio/structures/thermostat_parameters.py,sha256=M-VpUb7z-0rsES-PqUKpdmWP8TcV9IACkS6yPx4ivqo,6399
 pyplumio/structures/thermostat_sensors.py,sha256=yU5F2KXnpOMyHSdAfoIY6JmEIlSYMf05qmyvN0F4A5c,2236
-PyPlumIO-0.4.0.dist-info/LICENSE,sha256=g56wJgobsehVtkJD8MhM6isAnsdOUmRPv7fIaXI4Joo,1067
-PyPlumIO-0.4.0.dist-info/METADATA,sha256=BgWvmFmfiZc8_uo0U74HrcQ0NtH24cZOLKcxOkoIH88,4792
-PyPlumIO-0.4.0.dist-info/WHEEL,sha256=pkctZYzUS4AYVn6dJ-7367OJZivF2e8RA9b_ZBjif18,92
-PyPlumIO-0.4.0.dist-info/top_level.txt,sha256=kNBz9UPPkPD9teDn3U_sEy5LjzwLm9KfADCXtBlbw8A,9
-PyPlumIO-0.4.0.dist-info/RECORD,,
+PyPlumIO-0.4.1.dist-info/LICENSE,sha256=g56wJgobsehVtkJD8MhM6isAnsdOUmRPv7fIaXI4Joo,1067
+PyPlumIO-0.4.1.dist-info/METADATA,sha256=aUm-xRNBU8Yc7KumW9AmOLWd07CD6jMoIFZYvRC-3YA,4792
+PyPlumIO-0.4.1.dist-info/WHEEL,sha256=pkctZYzUS4AYVn6dJ-7367OJZivF2e8RA9b_ZBjif18,92
+PyPlumIO-0.4.1.dist-info/top_level.txt,sha256=kNBz9UPPkPD9teDn3U_sEy5LjzwLm9KfADCXtBlbw8A,9
+PyPlumIO-0.4.1.dist-info/RECORD,,

pyplumio/_version.py

@@ -1,4 +1,4 @@
 # file generated by setuptools_scm
 # don't change, don't track in version control
-__version__ = version = '0.4.0'
-__version_tuple__ = version_tuple = (0, 4, 0)
+__version__ = version = '0.4.1'
+__version_tuple__ = version_tuple = (0, 4, 1)

pyplumio/const.py

@@ -12,6 +12,7 @@ STATE_OFF: Final = "off"
 ATTR_CONNECTED: Final = "connected"
 ATTR_CURRENT_TEMP: Final = "current_temp"
 ATTR_DEVICE_INDEX: Final = "device_index"
+ATTR_FRAME_ERRORS: Final = "frame_errors"
 ATTR_INDEX: Final = "index"
 ATTR_LOADED: Final = "loaded"
 ATTR_OFFSET: Final = "offset"

pyplumio/devices/__init__.py

@@ -6,7 +6,7 @@ import logging
 from typing import ClassVar
 
 from pyplumio import util
-from pyplumio.const import ATTR_LOADED, DeviceType, FrameType
+from pyplumio.const import ATTR_FRAME_ERRORS, ATTR_LOADED, DeviceType, FrameType
 from pyplumio.exceptions import ParameterNotFoundError, UnknownDeviceError
 from pyplumio.frames import DataFrameDescription, Frame, Request, get_frame_handler
 from pyplumio.helpers.event_manager import EventManager
@@ -99,22 +99,23 @@ class Addressable(Device):
 
     async def async_setup(self) -> bool:
         """Setup addressable device object."""
-        try:
-            await asyncio.gather(
-                *{
-                    self.create_task(
-                        self.request(description.provides, description.frame_type)
-                    )
-                    for description in self._frame_types
-                },
-                return_exceptions=False,
-            )
-            await self.dispatch(ATTR_LOADED, True)
-            return True
-        except ValueError as e:
-            _LOGGER.error("Request failed: %s", e)
-            await self.dispatch(ATTR_LOADED, False)
-            return False
+        results = await asyncio.gather(
+            *{
+                self.create_task(
+                    self.request(description.provides, description.frame_type)
+                )
+                for description in self._frame_types
+            },
+            return_exceptions=True,
+        )
+
+        errors = [
+            result.args[1] for result in results if isinstance(result, ValueError)
+        ]
+
+        await self.dispatch(ATTR_FRAME_ERRORS, errors)
+        await self.dispatch(ATTR_LOADED, True)
+        return True
 
     async def request(
         self,
@@ -136,7 +137,7 @@ class Addressable(Device):
             except asyncio.TimeoutError:
                 retries -= 1
 
-        raise ValueError(f'could not request "{name}" with "{frame_type.name}"')
+        raise ValueError(f'could not request "{name}"', frame_type)
 
 
 class SubDevice(Device):

pyplumio/devices/ecomax.py

@@ -8,6 +8,7 @@ import time
 from typing import ClassVar, Final
 
 from pyplumio.const import (
+    ATTR_FRAME_ERRORS,
     ATTR_PASSWORD,
     ATTR_SENSORS,
     ATTR_STATE,
@@ -140,12 +141,24 @@ class EcoMAX(Addressable):
 
         super().handle_frame(frame)
 
+    def _has_frame_version(self, frame_type: FrameType | int, version: int) -> bool:
+        """Check if device instance has a version of the frame."""
+        return (
+            frame_type in self._frame_versions
+            and self._frame_versions[frame_type] == version
+        )
+
+    def _frame_is_supported(self, frame_type: FrameType | int) -> bool:
+        """Check if frame is supported by the device."""
+        return frame_type not in self.data.get(ATTR_FRAME_ERRORS, [])
+
     async def _update_frame_versions(self, versions: dict[int, int]) -> None:
         """Check versions and fetch outdated frames."""
         for frame_type, version in versions.items():
-            if is_known_frame_type(frame_type) and (
-                frame_type not in self._frame_versions
-                or self._frame_versions[frame_type] != version
+            if (
+                is_known_frame_type(frame_type)
+                and self._frame_is_supported(frame_type)
+                and not self._has_frame_version(frame_type, version)
             ):
                 # We don't have this frame or it's version has changed.
                 request = factory(get_frame_handler(frame_type), recipient=self.address)

pyplumio/structures/ecomax_parameters.py

@@ -10,13 +10,15 @@ from pyplumio.devices import Addressable
 from pyplumio.frames import Request
 from pyplumio.helpers.factory import factory
 from pyplumio.helpers.parameter import BinaryParameter, Parameter, ParameterDescription
-from pyplumio.helpers.typing import EventDataType, ParameterDataType, ParameterValueType
+from pyplumio.helpers.typing import EventDataType, ParameterValueType
 from pyplumio.structures import StructureDecoder, ensure_device_data
 from pyplumio.structures.thermostat_parameters import ATTR_THERMOSTAT_PROFILE
 
 ATTR_ECOMAX_CONTROL: Final = "ecomax_control"
 ATTR_ECOMAX_PARAMETERS: Final = "ecomax_parameters"
 
+ECOMAX_PARAMETER_SIZE: Final = 3
+
 
 class EcomaxParameter(Parameter):
     """Represents ecoMAX parameter."""
@@ -301,22 +303,33 @@ THERMOSTAT_PROFILE_PARAMETER = EcomaxParameterDescription(name=ATTR_THERMOSTAT_P
 class EcomaxParametersStructure(StructureDecoder):
     """Represents ecoMAX parameters data structure."""
 
+    _offset: int
+
+    def _ecomax_parameter(self, message: bytearray, start: int, end: int):
+        """Yields ecoMAX parameters."""
+        for index in range(start, start + end):
+            if parameter := util.unpack_parameter(message, self._offset):
+                yield (index, parameter)
+
+            self._offset += ECOMAX_PARAMETER_SIZE
+
     def decode(
         self, message: bytearray, offset: int = 0, data: EventDataType | None = None
     ) -> tuple[EventDataType, int]:
         """Decode bytes and return message data and offset."""
-        first_index = message[offset + 1]
-        last_index = message[offset + 2]
-        offset += 3
-        ecomax_parameters: list[tuple[int, ParameterDataType]] = []
-        for index in range(first_index, first_index + last_index):
-            parameter = util.unpack_parameter(message, offset)
-            if parameter is not None:
-                ecomax_parameters.append((index, parameter))
 
-            offset += 3
+        start = message[offset + 1]
+        end = message[offset + 2]
+        self._offset = offset + 3
 
         return (
-            ensure_device_data(data, {ATTR_ECOMAX_PARAMETERS: ecomax_parameters}),
-            offset,
+            ensure_device_data(
+                data,
+                {
+                    ATTR_ECOMAX_PARAMETERS: list(
+                        self._ecomax_parameter(message, start, end)
+                    )
+                },
+            ),
+            self._offset,
         )

pyplumio/structures/mixer_parameters.py

@@ -1,7 +1,6 @@
 """Contains mixer parameter structure decoder."""
 from __future__ import annotations
 
-from collections.abc import Iterable
 from dataclasses import dataclass
 from typing import TYPE_CHECKING, Final
 
@@ -18,6 +17,8 @@ if TYPE_CHECKING:
 
 ATTR_MIXER_PARAMETERS: Final = "mixer_parameters"
 
+MIXER_PARAMETER_SIZE: Final = 3
+
 
 class MixerParameter(Parameter):
     """Represents mixer parameter."""
@@ -117,48 +118,41 @@ ECOMAX_I_MIXER_PARAMETERS: tuple[MixerParameterDescription, ...] = (
 )
 
 
-def _decode_mixer_parameters(
-    message: bytearray, offset: int, indexes: Iterable
-) -> tuple[list[tuple[int, ParameterDataType]], int]:
-    """Decode parameters for a single mixer."""
-    parameters: list[tuple[int, ParameterDataType]] = []
-    for index in indexes:
-        parameter = util.unpack_parameter(message, offset)
-        if parameter is not None:
-            parameters.append((index, parameter))
+class MixerParametersStructure(StructureDecoder):
+    """Represent mixer parameters data structure."""
 
-        offset += 3
+    _offset: int
 
-    return parameters, offset
+    def _mixer_parameter(self, message: bytearray, start: int, end: int):
+        """Yields mixer parameters."""
+        for index in range(start, start + end):
+            if (parameter := util.unpack_parameter(message, self._offset)) is not None:
+                yield (index, parameter)
 
-
-class MixerParametersStructure(StructureDecoder):
-    """Represent mixer parameters data structure."""
+            self._offset += MIXER_PARAMETER_SIZE
 
     def decode(
         self, message: bytearray, offset: int = 0, data: EventDataType | None = None
     ) -> tuple[EventDataType, int]:
         """Decode bytes and return message data and offset."""
-        first_index = message[offset + 1]
-        last_index = message[offset + 2]
-        mixer_count = message[offset + 3]
-        parameter_count_per_mixer = first_index + last_index
-        offset += 4
+        start = message[offset + 1]
+        end = message[offset + 2]
+        mixers = message[offset + 3]
+        self._offset = offset + 4
+
         mixer_parameters: dict[int, list[tuple[int, ParameterDataType]]] = {}
-        for index in range(mixer_count):
-            parameters, offset = _decode_mixer_parameters(
-                message,
-                offset,
-                range(first_index, parameter_count_per_mixer),
-            )
-            if parameters:
-                mixer_parameters[index] = parameters
-
-        if not mixer_parameters:
-            # No mixer parameters detected.
-            return ensure_device_data(data, {ATTR_MIXER_PARAMETERS: None}), offset
+        for mixer in range(mixers):
+            if parameters := list(self._mixer_parameter(message, start, end)):
+                mixer_parameters[mixer] = parameters
 
         return (
-            ensure_device_data(data, {ATTR_MIXER_PARAMETERS: mixer_parameters}),
-            offset,
+            ensure_device_data(
+                data,
+                {
+                    ATTR_MIXER_PARAMETERS: (
+                        None if not mixer_parameters else mixer_parameters
+                    )
+                },
+            ),
+            self._offset,
         )

pyplumio/structures/thermostat_parameters.py

@@ -1,7 +1,6 @@
 """Contains thermostat parameter structure decoder."""
 from __future__ import annotations
 
-from collections.abc import Iterable
 from dataclasses import dataclass
 from typing import TYPE_CHECKING, Final
 
@@ -14,12 +13,15 @@ from pyplumio.helpers.typing import EventDataType, ParameterDataType, ParameterV
 from pyplumio.structures import StructureDecoder, ensure_device_data
 from pyplumio.structures.thermostat_sensors import ATTR_THERMOSTAT_COUNT
 
+if TYPE_CHECKING:
+    from pyplumio.devices.thermostat import Thermostat
+
+
 ATTR_THERMOSTAT_PROFILE: Final = "thermostat_profile"
 ATTR_THERMOSTAT_PARAMETERS: Final = "thermostat_parameters"
 ATTR_THERMOSTAT_PARAMETERS_DECODER: Final = "thermostat_parameters_decoder"
 
-if TYPE_CHECKING:
-    from pyplumio.devices.thermostat import Thermostat
+THERMOSTAT_PARAMETER_SIZE: Final = 3
 
 
 class ThermostatParameter(Parameter):
@@ -107,72 +109,68 @@ THERMOSTAT_PARAMETERS: tuple[ThermostatParameterDescription, ...] = (
 )
 
 
-def _decode_thermostat_parameters(
-    message: bytearray, offset: int, indexes: Iterable
-) -> tuple[list[tuple[int, ParameterDataType]], int]:
-    """Decode parameters for a single thermostat."""
-    parameters: list[tuple[int, ParameterDataType]] = []
-    for index in indexes:
-        description = THERMOSTAT_PARAMETERS[index]
-        parameter = util.unpack_parameter(message, offset, size=description.size)
-        if parameter is not None:
-            parameters.append((index, parameter))
-
-        offset += 3 * description.size
-
-    return parameters, offset
+def _empty_response(
+    offset: int, data: EventDataType | None = None
+) -> tuple[EventDataType, int]:
+    """Return empty response."""
+    return (
+        ensure_device_data(
+            data,
+            {ATTR_THERMOSTAT_PARAMETERS: None, ATTR_THERMOSTAT_PROFILE: None},
+        ),
+        offset,
+    )
 
 
 class ThermostatParametersStructure(StructureDecoder):
     """Represent thermostat parameters data structure."""
 
+    _offset: int
+
+    def _thermostat_parameter(
+        self, message: bytearray, thermostats: int, start: int, end: int
+    ):
+        """Yields thermostat parameters."""
+        for index in range(start, (start + end) // thermostats):
+            description = THERMOSTAT_PARAMETERS[index]
+            if (
+                parameter := util.unpack_parameter(
+                    message, self._offset, size=description.size
+                )
+            ) is not None:
+                yield (index, parameter)
+
+            self._offset += THERMOSTAT_PARAMETER_SIZE * description.size
+
     def decode(
         self, message: bytearray, offset: int = 0, data: EventDataType | None = None
     ) -> tuple[EventDataType, int]:
         """Decode bytes and return message data and offset."""
         data = ensure_device_data(data)
-        thermostat_count = data.get(ATTR_THERMOSTAT_COUNT, 0)
-        if thermostat_count == 0:
-            return (
-                ensure_device_data(
-                    data,
-                    {ATTR_THERMOSTAT_PARAMETERS: None, ATTR_THERMOSTAT_PROFILE: None},
-                ),
-                offset,
-            )
-
-        first_index = message[offset + 1]
-        last_index = message[offset + 2]
+        thermostats = data.get(ATTR_THERMOSTAT_COUNT, 0)
+        if thermostats == 0:
+            return _empty_response(offset, data)
+
+        start = message[offset + 1]
+        end = message[offset + 2]
         thermostat_profile = util.unpack_parameter(message, offset + 3)
-        parameter_count_per_thermostat = (first_index + last_index) // thermostat_count
-        offset += 6
+        self._offset = offset + 6
         thermostat_parameters: dict[int, list[tuple[int, ParameterDataType]]] = {}
-        for index in range(thermostat_count):
-            parameters, offset = _decode_thermostat_parameters(
-                message,
-                offset,
-                range(first_index, parameter_count_per_thermostat),
-            )
-            if parameters:
-                thermostat_parameters[index] = parameters
-
-        if not thermostat_parameters:
-            # No thermostat parameters detected.
-            return (
-                ensure_device_data(
-                    data,
-                    {ATTR_THERMOSTAT_PARAMETERS: None, ATTR_THERMOSTAT_PROFILE: None},
-                ),
-                offset,
-            )
+        for thermostat in range(thermostats):
+            if parameters := list(
+                self._thermostat_parameter(message, thermostats, start, end)
+            ):
+                thermostat_parameters[thermostat] = parameters
 
         return (
             ensure_device_data(
                 data,
                 {
                     ATTR_THERMOSTAT_PROFILE: thermostat_profile,
-                    ATTR_THERMOSTAT_PARAMETERS: thermostat_parameters,
+                    ATTR_THERMOSTAT_PARAMETERS: None
+                    if not thermostat_parameters
+                    else thermostat_parameters,
                 },
             ),
-            offset,
+            self._offset,
         )