PyPlumIO 0.5.34__tar.gz → 0.5.36__tar.gz

{pyplumio-0.5.34 → pyplumio-0.5.36}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: PyPlumIO
-Version: 0.5.34
+Version: 0.5.36
 Summary: PyPlumIO is a native ecoNET library for Plum ecoMAX controllers.
 Author-email: Denis Paavilainen <denpa@denpa.pro>
 License: MIT License
@@ -27,22 +27,22 @@ Requires-Dist: dataslots==1.2.0
 Requires-Dist: pyserial-asyncio==0.6
 Requires-Dist: typing-extensions==4.12.2
 Provides-Extra: test
-Requires-Dist: codespell==2.3.0; extra == "test"
+Requires-Dist: codespell==2.4.0; extra == "test"
 Requires-Dist: coverage==7.6.10; extra == "test"
 Requires-Dist: mypy==1.14.1; extra == "test"
 Requires-Dist: pyserial-asyncio-fast==0.14; extra == "test"
 Requires-Dist: pytest==8.3.4; extra == "test"
 Requires-Dist: pytest-asyncio==0.25.2; extra == "test"
-Requires-Dist: ruff==0.9.2; extra == "test"
-Requires-Dist: tox==4.23.2; extra == "test"
-Requires-Dist: types-pyserial==3.5.0.20241221; extra == "test"
+Requires-Dist: ruff==0.9.3; extra == "test"
+Requires-Dist: tox==4.24.1; extra == "test"
+Requires-Dist: types-pyserial==3.5.0.20250124; extra == "test"
 Provides-Extra: docs
 Requires-Dist: sphinx==8.1.3; extra == "docs"
 Requires-Dist: sphinx_rtd_theme==3.0.2; extra == "docs"
 Requires-Dist: readthedocs-sphinx-search==0.3.2; extra == "docs"
 Provides-Extra: dev
 Requires-Dist: pyplumio[docs,test]; extra == "dev"
-Requires-Dist: pre-commit==4.0.1; extra == "dev"
+Requires-Dist: pre-commit==4.1.0; extra == "dev"
 Requires-Dist: tomli==2.2.1; extra == "dev"
 
 # PyPlumIO is a native ecoNET library for Plum ecoMAX controllers.

{pyplumio-0.5.34 → pyplumio-0.5.36}/PyPlumIO.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: PyPlumIO
-Version: 0.5.34
+Version: 0.5.36
 Summary: PyPlumIO is a native ecoNET library for Plum ecoMAX controllers.
 Author-email: Denis Paavilainen <denpa@denpa.pro>
 License: MIT License
@@ -27,22 +27,22 @@ Requires-Dist: dataslots==1.2.0
 Requires-Dist: pyserial-asyncio==0.6
 Requires-Dist: typing-extensions==4.12.2
 Provides-Extra: test
-Requires-Dist: codespell==2.3.0; extra == "test"
+Requires-Dist: codespell==2.4.0; extra == "test"
 Requires-Dist: coverage==7.6.10; extra == "test"
 Requires-Dist: mypy==1.14.1; extra == "test"
 Requires-Dist: pyserial-asyncio-fast==0.14; extra == "test"
 Requires-Dist: pytest==8.3.4; extra == "test"
 Requires-Dist: pytest-asyncio==0.25.2; extra == "test"
-Requires-Dist: ruff==0.9.2; extra == "test"
-Requires-Dist: tox==4.23.2; extra == "test"
-Requires-Dist: types-pyserial==3.5.0.20241221; extra == "test"
+Requires-Dist: ruff==0.9.3; extra == "test"
+Requires-Dist: tox==4.24.1; extra == "test"
+Requires-Dist: types-pyserial==3.5.0.20250124; extra == "test"
 Provides-Extra: docs
 Requires-Dist: sphinx==8.1.3; extra == "docs"
 Requires-Dist: sphinx_rtd_theme==3.0.2; extra == "docs"
 Requires-Dist: readthedocs-sphinx-search==0.3.2; extra == "docs"
 Provides-Extra: dev
 Requires-Dist: pyplumio[docs,test]; extra == "dev"
-Requires-Dist: pre-commit==4.0.1; extra == "dev"
+Requires-Dist: pre-commit==4.1.0; extra == "dev"
 Requires-Dist: tomli==2.2.1; extra == "dev"
 
 # PyPlumIO is a native ecoNET library for Plum ecoMAX controllers.

{pyplumio-0.5.34 → pyplumio-0.5.36}/PyPlumIO.egg-info/requires.txt

@@ -4,7 +4,7 @@ typing-extensions==4.12.2
 
 [dev]
 pyplumio[docs,test]
-pre-commit==4.0.1
+pre-commit==4.1.0
 tomli==2.2.1
 
 [docs]
@@ -13,12 +13,12 @@ sphinx_rtd_theme==3.0.2
 readthedocs-sphinx-search==0.3.2
 
 [test]
-codespell==2.3.0
+codespell==2.4.0
 coverage==7.6.10
 mypy==1.14.1
 pyserial-asyncio-fast==0.14
 pytest==8.3.4
 pytest-asyncio==0.25.2
-ruff==0.9.2
-tox==4.23.2
-types-pyserial==3.5.0.20241221
+ruff==0.9.3
+tox==4.24.1
+types-pyserial==3.5.0.20250124

{pyplumio-0.5.34 → pyplumio-0.5.36}/pyplumio/__init__.py

@@ -101,6 +101,7 @@ __all__ = [
     "ProtocolError",
     "PyPlumIOError",
     "ReadError",
+    "RequestError",
     "SerialConnection",
     "TcpConnection",
     "UnknownDeviceError",

{pyplumio-0.5.34 → pyplumio-0.5.36}/pyplumio/_version.py

@@ -12,5 +12,5 @@ __version__: str
 __version_tuple__: VERSION_TUPLE
 version_tuple: VERSION_TUPLE
 
-__version__ = version = '0.5.34'
-__version_tuple__ = version_tuple = (0, 5, 34)
+__version__ = version = '0.5.36'
+__version_tuple__ = version_tuple = (0, 5, 36)

{pyplumio-0.5.34 → pyplumio-0.5.36}/pyplumio/connection.py

@@ -86,7 +86,7 @@ class Connection(ABC, TaskManager):
                 RECONNECT_TIMEOUT,
             )
             await asyncio.sleep(RECONNECT_TIMEOUT)
-            self.create_task(self._reconnect())
+            self.create_task(self._reconnect(), name="reconnect_task")
 
     async def connect(self) -> None:
         """Open the connection.

{pyplumio-0.5.34 → pyplumio-0.5.36}/pyplumio/devices/__init__.py

@@ -9,11 +9,11 @@ import logging
 from typing import Any, ClassVar
 
 from pyplumio.const import ATTR_FRAME_ERRORS, ATTR_LOADED, DeviceType, FrameType
-from pyplumio.exceptions import UnknownDeviceError
+from pyplumio.exceptions import RequestError, UnknownDeviceError
 from pyplumio.frames import DataFrameDescription, Frame, Request, is_known_frame_type
 from pyplumio.helpers.event_manager import EventManager
 from pyplumio.helpers.factory import create_instance
-from pyplumio.helpers.parameter import Parameter, ParameterValue
+from pyplumio.helpers.parameter import NumericType, Parameter, State
 from pyplumio.structures.frame_versions import ATTR_FRAME_VERSIONS
 from pyplumio.structures.network_info import NetworkInfo
 from pyplumio.utils import to_camelcase
@@ -57,7 +57,7 @@ class Device(ABC, EventManager):
     async def set(
         self,
         name: str,
-        value: ParameterValue,
+        value: NumericType | State | bool,
         retries: int = 5,
         timeout: float | None = None,
     ) -> bool:
@@ -89,7 +89,7 @@ class Device(ABC, EventManager):
     def set_nowait(
         self,
         name: str,
-        value: ParameterValue,
+        value: NumericType | State | bool,
         retries: int = 5,
         timeout: float | None = None,
     ) -> None:
@@ -180,7 +180,7 @@ class PhysicalDevice(Device, ABC):
         )
 
         errors = [
-            result.args[1] for result in results if isinstance(result, BaseException)
+            result.frame_type for result in results if isinstance(result, RequestError)
         ]
 
         await asyncio.gather(
@@ -203,9 +203,10 @@ class PhysicalDevice(Device, ABC):
             except asyncio.TimeoutError:
                 retries -= 1
 
-        raise ValueError(
-            f"Failed to request parameter '{name}' with frame type '{frame_type}' "
-            f"after {retries} retries."
+        raise RequestError(
+            f"Failed to request '{name}' with frame type '{frame_type}' after "
+            f"{retries} retries.",
+            frame_type=frame_type,
         )
 
     @classmethod

{pyplumio-0.5.34 → pyplumio-0.5.36}/pyplumio/devices/ecomax.py

@@ -59,7 +59,8 @@ ATTR_MIXERS: Final = "mixers"
 ATTR_THERMOSTATS: Final = "thermostats"
 ATTR_FUEL_BURNED: Final = "fuel_burned"
 
-MAX_TIME_SINCE_LAST_FUEL_UPDATE_NS: Final = 300 * 1000000000
+NANOSECONDS_IN_SECOND: Final = 1_000_000_000
+MAX_TIME_SINCE_LAST_FUEL_UPDATE_NS: Final = 300 * NANOSECONDS_IN_SECOND
 
 SETUP_FRAME_TYPES: tuple[DataFrameDescription, ...] = (
     DataFrameDescription(
@@ -105,16 +106,15 @@ class EcoMAX(PhysicalDevice):
     """Represents an ecoMAX controller."""
 
     address = DeviceType.ECOMAX
-
-    _fuel_burned_timestamp_ns: int
     _setup_frames = SETUP_FRAME_TYPES
 
+    _fuel_burned_time_ns: int
+
     def __init__(self, queue: asyncio.Queue[Frame], network: NetworkInfo) -> None:
         """Initialize a new ecoMAX controller."""
         super().__init__(queue, network)
-        self._fuel_burned_timestamp_ns = time.perf_counter_ns()
         self.subscribe(ATTR_ECOMAX_PARAMETERS, self._handle_ecomax_parameters)
-        self.subscribe(ATTR_FUEL_CONSUMPTION, self._add_burned_fuel_counter)
+        self.subscribe(ATTR_FUEL_CONSUMPTION, self._add_burned_fuel_meter)
         self.subscribe(ATTR_MIXER_PARAMETERS, self._handle_mixer_parameters)
         self.subscribe(ATTR_MIXER_SENSORS, self._handle_mixer_sensors)
         self.subscribe(ATTR_SCHEDULES, self._add_schedules)
@@ -124,6 +124,7 @@ class EcoMAX(PhysicalDevice):
         self.subscribe(ATTR_THERMOSTAT_PARAMETERS, self._handle_thermostat_parameters)
         self.subscribe(ATTR_THERMOSTAT_PROFILE, self._add_thermostat_profile_parameter)
         self.subscribe(ATTR_THERMOSTAT_SENSORS, self._handle_thermostat_sensors)
+        self._fuel_burned_time_ns = time.perf_counter_ns()
 
     async def async_setup(self) -> bool:
         """Set up an ecoMAX controller."""
@@ -183,13 +184,10 @@ class EcoMAX(PhysicalDevice):
                     description = ECOMAX_PARAMETERS[product.type][index]
                 except IndexError:
                     _LOGGER.warning(
-                        (
-                            "Encountered unknown ecoMAX parameter (%i): %s. "
-                            "Your device isn't fully compatible with this software and "
-                            "may not work properly. "
-                            "Please visit the issue tracker and open a feature "
-                            "request to support %s"
-                        ),
+                        "Encountered unknown ecoMAX parameter (%i): %s. "
+                        "Your device isn't fully compatible with this software "
+                        "and may not work properly. Please visit the issue tracker "
+                        "and open a feature request to support %s",
                         index,
                         values,
                         product.model,
@@ -211,19 +209,29 @@ class EcoMAX(PhysicalDevice):
         await asyncio.gather(*_ecomax_parameter_events())
         return True
 
-    async def _add_burned_fuel_counter(self, fuel_consumption: float) -> None:
-        """Calculate fuel burned since last sensor's data message."""
-        current_timestamp_ns = time.perf_counter_ns()
-        time_passed_ns = current_timestamp_ns - self._fuel_burned_timestamp_ns
-        self._fuel_burned_timestamp_ns = current_timestamp_ns
-        if time_passed_ns >= MAX_TIME_SINCE_LAST_FUEL_UPDATE_NS:
-            _LOGGER.warning(
-                "Skipping outdated fuel consumption data, was %i seconds old",
-                time_passed_ns / 1000000000,
+    async def _add_burned_fuel_meter(self, fuel_consumption: float) -> None:
+        """Calculate and dispatch the amount of fuel burned.
+
+        This method calculates the fuel burned based on the time
+        elapsed since the last sensor message, which contains fuel
+        consumption data. If the elapsed time is within the acceptable
+        range, it dispatches the fuel burned data. Otherwise, it logs a
+        warning and skips the outdated data.
+        """
+        time_ns = time.perf_counter_ns()
+        nanoseconds_passed = time_ns - self._fuel_burned_time_ns
+        self._fuel_burned_time_ns = time_ns
+        if nanoseconds_passed < MAX_TIME_SINCE_LAST_FUEL_UPDATE_NS:
+            return self.dispatch_nowait(
+                ATTR_FUEL_BURNED,
+                fuel_consumption * nanoseconds_passed / (3600 * NANOSECONDS_IN_SECOND),
             )
-        else:
-            fuel_burned = fuel_consumption * time_passed_ns / (3600 * 1000000000)
-            await self.dispatch(ATTR_FUEL_BURNED, fuel_burned)
+
+        _LOGGER.warning(
+            "Skipping outdated fuel consumption data: %f (was %i seconds old)",
+            fuel_consumption,
+            nanoseconds_passed / NANOSECONDS_IN_SECOND,
+        )
 
     async def _handle_mixer_parameters(
         self,

{pyplumio-0.5.34 → pyplumio-0.5.36}/pyplumio/devices/mixer.py

@@ -64,13 +64,10 @@ class Mixer(VirtualDevice):
                     description = MIXER_PARAMETERS[product.type][index]
                 except IndexError:
                     _LOGGER.warning(
-                        (
-                            "Encountered unknown mixer parameter (%i): %s. "
-                            "Your device isn't fully compatible with this software and "
-                            "may not work properly. "
-                            "Please visit the issue tracker and open a feature "
-                            "request to support %s"
-                        ),
+                        "Encountered unknown mixer parameter (%i): %s. "
+                        "Your device isn't fully compatible with this software "
+                        "and may not work properly. Please visit the issue tracker "
+                        "and open a feature request to support %s",
                         index,
                         values,
                         product.model,

{pyplumio-0.5.34 → pyplumio-0.5.36}/pyplumio/exceptions.py

@@ -2,6 +2,8 @@
 
 from __future__ import annotations
 
+from pyplumio.const import FrameType
+
 
 class PyPlumIOError(Exception):
     """Base PyPlumIO error class."""
@@ -11,6 +13,15 @@ class ConnectionFailedError(PyPlumIOError):
     """Raised on connection failure."""
 
 
+class RequestError(PyPlumIOError):
+    """Raised on request error."""
+
+    def __init__(self, message: str, frame_type: FrameType) -> None:
+        """Initialize a new RequestError."""
+        super().__init__(message)
+        self.frame_type = frame_type
+
+
 class ProtocolError(PyPlumIOError):
     """Base class for protocol-related errors."""
 

{pyplumio-0.5.34 → pyplumio-0.5.36}/pyplumio/helpers/parameter.py

@@ -6,7 +6,7 @@ from abc import ABC, abstractmethod
 import asyncio
 from dataclasses import dataclass
 import logging
-from typing import TYPE_CHECKING, Any, Literal, TypeVar, Union
+from typing import TYPE_CHECKING, Any, Literal, TypeVar, Union, get_args
 
 from dataslots import dataslots
 from typing_extensions import TypeAlias
@@ -19,8 +19,8 @@ if TYPE_CHECKING:
 
 _LOGGER = logging.getLogger(__name__)
 
-
-ParameterValue: TypeAlias = Union[int, float, bool, Literal["off", "on"]]
+NumericType: TypeAlias = Union[int, float]
+State: TypeAlias = Literal["on", "off"]
 ParameterT = TypeVar("ParameterT", bound="Parameter")
 
 
@@ -28,7 +28,7 @@ def unpack_parameter(
     data: bytearray, offset: int = 0, size: int = 1
 ) -> ParameterValues | None:
     """Unpack a device parameter."""
-    if not check_parameter(data[offset : offset + size * 3]):
+    if not validate_parameter(data[offset : offset + size * 3]):
         return None
 
     value = data[offset : offset + size]
@@ -42,14 +42,17 @@ def unpack_parameter(
     )
 
 
-def check_parameter(data: bytearray) -> bool:
+def validate_parameter(data: bytearray) -> bool:
     """Check if parameter contains any bytes besides 0xFF."""
     return any(x for x in data if x != BYTE_UNDEFINED)
 
 
-def _normalize_parameter_value(value: ParameterValue) -> int:
-    """Normalize a parameter value."""
-    if value in (STATE_OFF, STATE_ON):
+def parameter_value_to_int(value: NumericType | State | bool) -> int:
+    """Convert a parameter value to an integer.
+
+    If the value is STATE_OFF or STATE_ON, it returns 0 or 1 respectively.
+    """
+    if value in get_args(State):
         return 1 if value == STATE_ON else 0
 
     return int(value)
@@ -77,19 +80,11 @@ class ParameterDescription:
 class Parameter(ABC):
     """Represents a base parameter."""
 
-    __slots__ = (
-        "device",
-        "description",
-        "_pending_update",
-        "_previous_value",
-        "_index",
-        "_values",
-    )
+    __slots__ = ("device", "description", "_pending_update", "_index", "_values")
 
     device: Device
     description: ParameterDescription
     _pending_update: bool
-    _previous_value: int
     _index: int
     _values: ParameterValues
 
@@ -103,8 +98,8 @@ class Parameter(ABC):
         """Initialize a new parameter."""
         self.device = device
         self.description = description
+        self._index = index
         self._pending_update = False
-        self._previous_value = 0
         self._index = index
         self._values = values if values else ParameterValues(0, 0, 0)
 
@@ -127,9 +122,9 @@ class Parameter(ABC):
             handler = getattr(self.values, method_to_call)
             return handler(other)
 
-        if isinstance(other, (int, float, bool)) or other in (STATE_OFF, STATE_ON):
+        if isinstance(other, (int, float, bool)) or other in get_args(State):
             handler = getattr(self.values.value, method_to_call)
-            return handler(_normalize_parameter_value(other))
+            return handler(parameter_value_to_int(other))
         else:
             return NotImplemented
 
@@ -184,42 +179,44 @@ class Parameter(ABC):
         )
         return type(self)(self.device, self.description, values)
 
-    def validate(self, value: ParameterValue) -> int:
+    def validate(self, value: NumericType | State | bool) -> int:
         """Validate a parameter value."""
-        value = _normalize_parameter_value(value)
-        if value < self.values.min_value or value > self.values.max_value:
+        int_value = parameter_value_to_int(value)
+        if int_value < self.values.min_value or int_value > self.values.max_value:
             raise ValueError(
                 f"Invalid value: {value}. Must be between "
                 f"{self.min_value} and {self.max_value}."
             )
 
-        return value
+        return int_value
 
     async def set(self, value: Any, retries: int = 5, timeout: float = 5.0) -> bool:
         """Set a parameter value."""
-        return await self._try_set(self.validate(value), retries, timeout)
+        return await self._attempt_update(self.validate(value), retries, timeout)
 
     def set_nowait(self, value: Any, retries: int = 5, timeout: float = 5.0) -> None:
         """Set a parameter value without waiting."""
-        self.device.create_task(self._try_set(self.validate(value), retries, timeout))
+        self.device.create_task(
+            self._attempt_update(self.validate(value), retries, timeout)
+        )
 
-    async def _try_set(
-        self, value: Any, retries: int = 5, timeout: float = 5.0
+    async def _attempt_update(
+        self, value: int, retries: int = 5, timeout: float = 5.0
     ) -> bool:
-        """Try to set a parameter value."""
+        """Attempt to update a parameter value on the remote device."""
         if value == self.values.value:
             # Value is unchanged
             return True
 
-        self._previous_value = self._values.value
-        self._values.value = value
         self._pending_update = True
+        self._values.value = value
+        initial_retries = retries
         while self.pending_update:
             if retries <= 0:
                 _LOGGER.warning(
-                    "Failed to set parameter '%s' after %d retries",
+                    "Unable to confirm update of '%s' parameter after %d retries",
                     self.description.name,
-                    retries,
+                    initial_retries,
                 )
                 return False
 
@@ -231,9 +228,7 @@ class Parameter(ABC):
 
     def update(self, values: ParameterValues) -> None:
         """Update the parameter values."""
-        if self.pending_update and self._previous_value != values.value:
-            self._pending_update = False
-
+        self._pending_update = False
         self._values = values
 
     @property
@@ -267,17 +262,17 @@ class Parameter(ABC):
 
     @property
     @abstractmethod
-    def value(self) -> Any:
+    def value(self) -> NumericType | State | bool:
         """Return the value."""
 
     @property
     @abstractmethod
-    def min_value(self) -> Any:
+    def min_value(self) -> NumericType | State | bool:
         """Return the minimum allowed value."""
 
     @property
     @abstractmethod
-    def max_value(self) -> Any:
+    def max_value(self) -> NumericType | State | bool:
         """Return the maximum allowed value."""
 
     @abstractmethod
@@ -301,13 +296,13 @@ class Number(Parameter):
     description: NumberDescription
 
     async def set(
-        self, value: int | float, retries: int = 5, timeout: float = 5.0
+        self, value: NumericType, retries: int = 5, timeout: float = 5.0
     ) -> bool:
         """Set a parameter value."""
         return await super().set(value, retries, timeout)
 
     def set_nowait(
-        self, value: int | float, retries: int = 5, timeout: float = 5.0
+        self, value: NumericType, retries: int = 5, timeout: float = 5.0
     ) -> None:
         """Set a parameter value without waiting."""
         super().set_nowait(value, retries, timeout)
@@ -317,17 +312,17 @@ class Number(Parameter):
         return Request()
 
     @property
-    def value(self) -> int | float:
+    def value(self) -> NumericType:
         """Return the value."""
         return self.values.value
 
     @property
-    def min_value(self) -> int | float:
+    def min_value(self) -> NumericType:
         """Return the minimum allowed value."""
         return self.values.min_value
 
     @property
-    def max_value(self) -> int | float:
+    def max_value(self) -> NumericType:
         """Return the maximum allowed value."""
         return self.values.max_value
 
@@ -351,13 +346,13 @@ class Switch(Parameter):
     description: SwitchDescription
 
     async def set(
-        self, value: bool | Literal["off", "on"], retries: int = 5, timeout: float = 5.0
+        self, value: State | bool, retries: int = 5, timeout: float = 5.0
     ) -> bool:
         """Set a parameter value."""
         return await super().set(value, retries, timeout)
 
     def set_nowait(
-        self, value: bool | Literal["off", "on"], retries: int = 5, timeout: float = 5.0
+        self, value: State | bool, retries: int = 5, timeout: float = 5.0
     ) -> None:
         """Set a switch value without waiting."""
         super().set_nowait(value, retries, timeout)
@@ -393,7 +388,7 @@ class Switch(Parameter):
         return Request()
 
     @property
-    def value(self) -> Literal["off", "on"]:
+    def value(self) -> State:
         """Return the value."""
         return STATE_ON if self.values.value == 1 else STATE_OFF
 

{pyplumio-0.5.34 → pyplumio-0.5.36}/pyplumio/helpers/schedule.py

@@ -21,8 +21,7 @@ TIME_FORMAT: Final = "%H:%M"
 STATE_NIGHT: Final = "night"
 STATE_DAY: Final = "day"
 
-ON_STATES: Final = (STATE_ON, STATE_DAY)
-OFF_STATES: Final = (STATE_OFF, STATE_NIGHT)
+_ON_STATES: Final = {STATE_ON, STATE_DAY}
 
 ScheduleState: TypeAlias = Literal["on", "off", "day", "night"]
 Time = Annotated[str, "time in HH:MM format"]
@@ -113,7 +112,7 @@ class ScheduleDay(MutableMapping):
             )
 
         for index in _get_time_range(start, end):
-            self._intervals[index] = True if state in ON_STATES else False
+            self._intervals[index] = True if state in _ON_STATES else False
 
     def set_on(self, start: Time = "00:00", end: Time = "00:00") -> None:
         """Set a schedule interval state to 'on'."""

{pyplumio-0.5.34 → pyplumio-0.5.36}/pyplumio/helpers/uid.py

@@ -1,4 +1,4 @@
-"""Contains an UID helpers."""
+"""Contains UID helpers."""
 
 from __future__ import annotations
 
@@ -18,18 +18,18 @@ def decode_uid(buffer: bytes) -> str:
 def _base5(buffer: bytes) -> str:
     """Encode bytes to a base5 encoded string."""
     number = int.from_bytes(buffer, byteorder="little")
-    output = ""
+    output = []
     while number:
-        output = BASE5_KEY[number & 0b00011111] + output
+        output.append(BASE5_KEY[number & 0b00011111])
         number >>= 5
 
-    return output
+    return "".join(reversed(output))
 
 
 def _crc16(buffer: bytes) -> bytes:
     """Return a CRC 16."""
     crc16 = reduce(_crc16_byte, buffer, CRC)
-    return crc16.to_bytes(byteorder="little", length=2)
+    return crc16.to_bytes(length=2, byteorder="little")
 
 
 def _crc16_byte(crc: int, byte: int) -> int:

{pyplumio-0.5.34 → pyplumio-0.5.36}/pyproject.toml

@@ -37,15 +37,15 @@ dynamic = ["version"]
 
 [project.optional-dependencies]
 test = [
-    "codespell==2.3.0",
+    "codespell==2.4.0",
     "coverage==7.6.10",
     "mypy==1.14.1",
     "pyserial-asyncio-fast==0.14",
     "pytest==8.3.4",
     "pytest-asyncio==0.25.2",
-    "ruff==0.9.2",
-    "tox==4.23.2",
-    "types-pyserial==3.5.0.20241221"
+    "ruff==0.9.3",
+    "tox==4.24.1",
+    "types-pyserial==3.5.0.20250124"
 ]
 docs = [
     "sphinx==8.1.3",
@@ -54,7 +54,7 @@ docs = [
 ]
 dev = [
     "pyplumio[test,docs]",
-    "pre-commit==4.0.1",
+    "pre-commit==4.1.0",
     "tomli==2.2.1"
 ]
 

{pyplumio-0.5.34 → pyplumio-0.5.36}/requirements_test.txt

@@ -1,11 +1,11 @@
-codespell==2.3.0
+codespell==2.4.0
 coverage==7.6.10
 mypy==1.14.1
-pre-commit==4.0.1
+pre-commit==4.1.0
 pyserial-asyncio-fast==0.14
 pytest-asyncio==0.25.2
 pytest==8.3.4
-ruff==0.9.2
+ruff==0.9.3
 tomli==2.2.1
-tox==4.23.2
-types-pyserial==3.5.0.20241221
+tox==4.24.1
+types-pyserial==3.5.0.20250124