PyPlumIO 0.4.0.post1__py3-none-any.whl → 0.4.2__py3-none-any.whl

{PyPlumIO-0.4.0.post1.dist-info → PyPlumIO-0.4.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: PyPlumIO
-Version: 0.4.0.post1
+Version: 0.4.2
 Summary: PyPlumIO is a native ecoNET library for Plum ecoMAX controllers.
 Author-email: Denis Paavilainen <denpa@denpa.pro>
 License: MIT License

{PyPlumIO-0.4.0.post1.dist-info → PyPlumIO-0.4.2.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=YOs6bNRc9Kz_3q_WwUn40ymWrAd7HGLCakgxVcAYoqg,166
+pyplumio/_version.py,sha256=MHz4IpMPBfgMjF4EUb11AphGoLXSC-lOtHXxzKzJNjk,160
 pyplumio/connection.py,sha256=ABeJFbnqTq89i5J1TRR2sdQeRrq1Pewdi_TaDjzSOOI,5494
-pyplumio/const.py,sha256=WfV68bd99IWsKHQ2bWW9Rl8xHvgfWrt8cyb2kjs5od4,2965
+pyplumio/const.py,sha256=qCsgmv-uIjLBae46AlMMPPHxag301LJaRTUo9NLmU2M,3039
 pyplumio/exceptions.py,sha256=cpLGRl9db6cU1Z6WXhQ7yl1V7Te193QymWiOoRF8KAI,798
-pyplumio/filters.py,sha256=X_Q_DE9M_yyw2OWyr_-IargtAWpW9CAI3jIhoIXKhaA,5648
+pyplumio/filters.py,sha256=MM3l67Iq0Xm2aZRPkAOAxoeaMT3cTsF-vL95Sid4GTM,6548
 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
@@ -25,19 +25,19 @@ pyplumio/helpers/parameter.py,sha256=4PxOQg0jFAPwrdQkwLXjaey-CgDCGsNCcglldk2zwm4
 pyplumio/helpers/schedule.py,sha256=pbKRqcU48IRrEKZQOncBQkz_WExxjtDDHZ3x-MHpVIg,4077
 pyplumio/helpers/task_manager.py,sha256=fhqbSNaInoWXL3YjkFe_sCgh8suOwRnufqWBrXIgPeM,1026
 pyplumio/helpers/timeout.py,sha256=az12syvMGhTxzeiLGZIcJYG6MI4t6LCZeZyt6viRh04,846
-pyplumio/helpers/typing.py,sha256=krNU_6ZDgWc6A30K-N-_2Tbgj4SiERsAC53EN5NXCQs,365
+pyplumio/helpers/typing.py,sha256=wDy-TLxLAF_jrw8QW5g4EZpbV8unRYqH_VO8s6VnIYE,557
 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.post1.dist-info/LICENSE,sha256=g56wJgobsehVtkJD8MhM6isAnsdOUmRPv7fIaXI4Joo,1067
-PyPlumIO-0.4.0.post1.dist-info/METADATA,sha256=yMNNaD1R8pgjDG2uk7S55la2KMmKkN3_PV0lhhmvlpQ,4798
-PyPlumIO-0.4.0.post1.dist-info/WHEEL,sha256=pkctZYzUS4AYVn6dJ-7367OJZivF2e8RA9b_ZBjif18,92
-PyPlumIO-0.4.0.post1.dist-info/top_level.txt,sha256=kNBz9UPPkPD9teDn3U_sEy5LjzwLm9KfADCXtBlbw8A,9
-PyPlumIO-0.4.0.post1.dist-info/RECORD,,
+PyPlumIO-0.4.2.dist-info/LICENSE,sha256=g56wJgobsehVtkJD8MhM6isAnsdOUmRPv7fIaXI4Joo,1067
+PyPlumIO-0.4.2.dist-info/METADATA,sha256=8iQtMCrvNMnHyiN7p0q3XjcSR6sbljW4FEQJ3rjF0Eo,4792
+PyPlumIO-0.4.2.dist-info/WHEEL,sha256=pkctZYzUS4AYVn6dJ-7367OJZivF2e8RA9b_ZBjif18,92
+PyPlumIO-0.4.2.dist-info/top_level.txt,sha256=kNBz9UPPkPD9teDn3U_sEy5LjzwLm9KfADCXtBlbw8A,9
+PyPlumIO-0.4.2.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.post1'
-__version_tuple__ = version_tuple = (0, 4, 0)
+__version__ = version = '0.4.2'
+__version_tuple__ = version_tuple = (0, 4, 2)

pyplumio/const.py

@@ -4,6 +4,8 @@ from __future__ import annotations
 from enum import IntEnum, unique
 from typing import Final
 
+UNDEFINED: Final = "undefined"
+
 # Binary states.
 STATE_ON: Final = "on"
 STATE_OFF: Final = "off"
@@ -12,6 +14,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/filters.py

@@ -4,34 +4,70 @@ from __future__ import annotations
 from abc import ABC, abstractmethod
 import math
 import time
-from typing import Any, Final
+from typing import Any, Final, SupportsFloat, SupportsIndex, overload
 
+from pyplumio.const import UNDEFINED
 from pyplumio.helpers.parameter import Parameter
-from pyplumio.helpers.typing import EventCallbackType
+from pyplumio.helpers.typing import EventCallbackType, SupportsSubtraction
 
 TOLERANCE: Final = 0.1
 
 
-def _significantly_changed(old_value, new_value) -> bool:
+@overload
+def _significantly_changed(old: Parameter, new: Parameter) -> bool:
+    """Check if parameter is significantly changed."""
+
+
+@overload
+def _significantly_changed(
+    old: SupportsFloat | SupportsIndex, new: SupportsFloat | SupportsIndex
+) -> bool:
+    """Check if float value is significantly changed."""
+
+
+def _significantly_changed(old, new) -> bool:
     """Check if value is significantly changed."""
-    if old_value is None or (isinstance(old_value, Parameter) and old_value.is_changed):
+    if old == UNDEFINED:
         return True
 
+    if isinstance(old, Parameter) and old.is_changed:
+        return True
+
+    if isinstance(old, Parameter) and isinstance(new, Parameter):
+        return (
+            old.value != new.value
+            or old.min_value != new.min_value
+            or old.max_value != new.max_value
+        )
+
     try:
-        return not math.isclose(old_value, new_value, abs_tol=TOLERANCE)
+        return not math.isclose(old, new, abs_tol=TOLERANCE)
     except TypeError:
         pass
 
-    return old_value != new_value
+    return old != new
+
+
+@overload
+def _diffence_between(old: list, new: list) -> list:
+    """Return the difference between lists."""
 
 
-def _diffence_between(old_value, new_value):
+@overload
+def _diffence_between(old: SupportsSubtraction, new: SupportsSubtraction) -> list:
+    """Return the difference between lists."""
+
+
+def _diffence_between(old, new):
     """Return the difference between values."""
-    if isinstance(old_value, list) and isinstance(new_value, list):
-        return [x for x in new_value if x not in old_value]
+    if old == UNDEFINED:
+        return None
+
+    if isinstance(old, list) and isinstance(new, list):
+        return [x for x in new if x not in old]
 
-    if hasattr(old_value, "__sub__") and hasattr(new_value, "__sub__"):
-        return new_value - old_value
+    if hasattr(old, "__sub__") and hasattr(new, "__sub__"):
+        return new - old
 
     return None
 
@@ -40,12 +76,12 @@ class Filter(ABC):
     """Represents base for value callback modifiers."""
 
     _callback: Any
-    _value: Any
+    _value: Any = UNDEFINED
 
     def __init__(self, callback: EventCallbackType):
         """Initialize new Filter object."""
         self._callback = callback
-        self._value = None
+        self._value = UNDEFINED
 
     def __eq__(self, other) -> bool:
         """Compare debounced callbacks."""
@@ -59,21 +95,21 @@ class Filter(ABC):
 
     @abstractmethod
     async def __call__(self, new_value):
-        """Set new value for the callback."""
+        """Set a new value for the callback."""
 
 
 class _OnChange(Filter):
     """Provides changed functionality to the callback."""
 
     async def __call__(self, new_value):
-        """Set new value for the callback."""
+        """Set a new value for the callback."""
         if _significantly_changed(self._value, new_value):
             self._value = new_value
             return await self._callback(new_value)
 
 
 def on_change(callback: EventCallbackType) -> _OnChange:
-    """Helper for change callback filter."""
+    """Helper for a change callback filter."""
     return _OnChange(callback)
 
 
@@ -90,20 +126,20 @@ class _Debounce(Filter):
         self._min_calls = min_calls
 
     async def __call__(self, new_value):
-        """Set new value for the callback."""
+        """Set a new value for the callback."""
         if _significantly_changed(self._value, new_value):
             self._calls += 1
         else:
             self._calls = 0
 
-        if self._calls >= self._min_calls or self._value is None:
+        if self._value == UNDEFINED or self._calls >= self._min_calls:
             self._value = new_value
             self._calls = 0
             return await self._callback(new_value)
 
 
 def debounce(callback: EventCallbackType, min_calls) -> _Debounce:
-    """Helper method for debounce callback filter."""
+    """Helper method for a debounce callback filter."""
     return _Debounce(callback, min_calls)
 
 
@@ -120,7 +156,7 @@ class _Throttle(Filter):
         self._timeout = seconds
 
     async def __call__(self, new_value):
-        """set new value for the callback."""
+        """Set a new value for the callback."""
         current_timestamp = time.monotonic()
         if (
             self._last_called is None
@@ -131,7 +167,7 @@ class _Throttle(Filter):
 
 
 def throttle(callback: EventCallbackType, seconds: float) -> _Throttle:
-    """Helper method for throttle callback filter."""
+    """Helper method for a throttle callback filter."""
     return _Throttle(callback, seconds)
 
 
@@ -139,16 +175,16 @@ class _Delta(Filter):
     """Provides ability to pass call difference to the callback."""
 
     async def __call__(self, new_value):
-        """set new value for the callback."""
-        old_value = self._value
-        if _significantly_changed(old_value, new_value):
+        """Set new value for the callback."""
+        if _significantly_changed(self._value, new_value):
+            old_value = self._value
             self._value = new_value
             if (difference := _diffence_between(old_value, new_value)) is not None:
                 return await self._callback(difference)
 
 
 def delta(callback: EventCallbackType) -> _Delta:
-    """Helper method for delta callback filter."""
+    """Helper method for a delta callback filter."""
     return _Delta(callback)
 
 
@@ -168,7 +204,7 @@ class _Aggregate(Filter):
         self._sum = 0.0
 
     async def __call__(self, new_value):
-        """Set new value for the callback."""
+        """Set a new value for the callback."""
         current_timestamp = time.monotonic()
         try:
             self._sum += new_value
@@ -185,5 +221,5 @@ class _Aggregate(Filter):
 
 
 def aggregate(callback: EventCallbackType, seconds: float) -> _Aggregate:
-    """Helper method for total callback filter."""
+    """Helper method for a total callback filter."""
     return _Aggregate(callback, seconds)

pyplumio/helpers/typing.py

@@ -2,9 +2,17 @@
 from __future__ import annotations
 
 from collections.abc import Awaitable, Callable
-from typing import Any, Literal, Union
+from typing import Any, Literal, Protocol, Union
 
 ParameterDataType = tuple[int, int, int]
 ParameterValueType = Union[int, float, bool, Literal["off"], Literal["on"]]
 EventDataType = dict[Union[str, int], Any]
 EventCallbackType = Callable[[Any], Awaitable[Any]]
+UndefinedType = Literal["undefined"]
+
+
+class SupportsSubtraction(Protocol):
+    """Supports subtraction operation."""
+
+    def __sub__(self, other):
+        """Subtracts a value."""

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,
         )