aioWiserHeatAPI 1.5.10__tar.gz → 1.5.11__tar.gz

{aioWiserHeatAPI-1.5.10 → aioWiserHeatAPI-1.5.11}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: aioWiserHeatAPI
-Version: 1.5.10
+Version: 1.5.11
 Summary: An AsyncIO API for controlling the Drayton Wiser Heating system
 Home-page: https://github.com/msp1974/aioWiserHeatAPI
 Author: Mark Parker
@@ -18,7 +18,7 @@ Requires-Dist: aiohttp>=3.9.3
 Requires-Dist: aiofiles>=23.2.1
 Requires-Dist: pyyaml>=6.0.1
 
-# Drayton Wiser Hub API Async v1.5.10
+# Drayton Wiser Hub API Async v1.5.11
 
 This repository contains a simple API which queries the Drayton Wiser Heating sysystem used in the UK.
 
@@ -72,6 +72,11 @@ Documentation available in [info.md](https://github.com/msp1974/wiserHeatAPIv2/b
 
 ## Changelog
 
+### v1.5.11
+
+* Improve error management
+* Add additional debug logging
+
 ### v1.5.10
 
 * Add retry routine to prevent hub response errors

{aioWiserHeatAPI-1.5.10 → aioWiserHeatAPI-1.5.11}/README.md

@@ -1,4 +1,4 @@
-# Drayton Wiser Hub API Async v1.5.10
+# Drayton Wiser Hub API Async v1.5.11
 
 This repository contains a simple API which queries the Drayton Wiser Heating sysystem used in the UK.
 
@@ -52,6 +52,11 @@ Documentation available in [info.md](https://github.com/msp1974/wiserHeatAPIv2/b
 
 ## Changelog
 
+### v1.5.11
+
+* Improve error management
+* Add additional debug logging
+
 ### v1.5.10
 
 * Add retry routine to prevent hub response errors

{aioWiserHeatAPI-1.5.10 → aioWiserHeatAPI-1.5.11}/aioWiserHeatAPI/__init__.py

@@ -3,6 +3,6 @@ import logging
 name = "aioWiserHeatAPI"
 __all__ = ["wiserAPI", "wiserDiscovery"]
 
-__VERSION__ = "1.5.10"
+__VERSION__ = "1.5.11"
 
 _LOGGER = logging.getLogger(__name__)

{aioWiserHeatAPI-1.5.10 → aioWiserHeatAPI-1.5.11}/aioWiserHeatAPI/const.py

@@ -30,6 +30,7 @@ TRV_BATTERY_LEVEL_MAPPING = {
 
 
 # Other Constants
+REST_RETRY_BACKOFF = [0.5, 1, 2, 3, 5]
 REST_BACKOFF_FACTOR = 1
 REST_RETRIES = 3
 REST_TIMEOUT = 10

{aioWiserHeatAPI-1.5.10 → aioWiserHeatAPI-1.5.11}/aioWiserHeatAPI/rest_controller.py

@@ -5,12 +5,14 @@ import enum
 import json
 import re
 from dataclasses import dataclass
+from datetime import datetime
 from typing import Optional
 
 import aiohttp
 
 from . import _LOGGER
 from .const import (
+    REST_RETRY_BACKOFF,
     REST_TIMEOUT,
     WISERHUBDOMAIN,
     WISERHUBSCHEDULES,
@@ -84,7 +86,6 @@ class _WiserRestController(object):
         raise_for_endpoint_error: bool = True,
     ):
         """Function to retry on response errors due to inconsistant isues reading from the hub."""
-        backoff = [1, 2, 3, 5, 10]
         http_version = aiohttp.HttpVersion11
         for i in range(0, 5):
             if i > 2:
@@ -101,9 +102,11 @@ class _WiserRestController(object):
                 return response
             except WiserHubResponseError as ex:
                 # Last 2 attempts try http 1.0
-                _LOGGER.debug("%s. Retrying in %is", ex, backoff[i])
+                _LOGGER.debug(
+                    "%s. Retrying in %.1fs", ex, REST_RETRY_BACKOFF[i]
+                )
                 # Backing off pause
-                await asyncio.sleep(backoff[i])
+                await asyncio.sleep(REST_RETRY_BACKOFF[i])
 
         raise WiserHubConnectionError(
             f"Unable to get a valid response from the Wiser hub. "
@@ -134,7 +137,6 @@ class _WiserRestController(object):
         kwargs = {}
         kwargs["headers"] = {
             "SECRET": self._wiser_connection_info.secret,
-            "Connection": "close",
         }
 
         # if data is not None:
@@ -232,7 +234,7 @@ class _WiserRestController(object):
                 f"Error code is: {response.status}"
             )
 
-    async def _get_hub_data(
+    async def get_hub_data(
         self, url: str, raise_for_endpoint_error: bool = True
     ):
         """Get data from hub"""
@@ -242,7 +244,7 @@ class _WiserRestController(object):
             raise_for_endpoint_error=raise_for_endpoint_error,
         )
 
-    async def _get_extra_config_data(self):
+    async def get_extra_config_data(self):
         # Load extra config file
 
         if self._extra_config_file:

{aioWiserHeatAPI-1.5.10 → aioWiserHeatAPI-1.5.11}/aioWiserHeatAPI/wiserhub.py

@@ -12,13 +12,10 @@ https://github.com/asantaga/wiserheatingapi
 This API allows you to get information from and control your wiserhub.
 """
 
-# TODO: Keep objects and update instead of recreating on hub update
-# TODO: Update entity values after commend issued to get current values
 import pathlib
+from datetime import datetime
 from typing import Optional
 
-import aiohttp
-
 from . import __VERSION__, _LOGGER
 from .cli import log_response_to_file
 from .const import (
@@ -121,6 +118,7 @@ class WiserAPI:
             )
 
     async def read_hub_data(self):
+        """Update data objects form the hub."""
         await self._build_objects()
 
         # Run automations
@@ -131,119 +129,150 @@ class WiserAPI:
             if await automations.run_automations():
                 await self._build_objects()
 
-    async def _build_objects(self):
-        """Read all data from hub and populate objects"""
-
-        # Read data from hub
-        self._domain_data = await self._wiser_rest_controller._get_hub_data(
-            WISERHUBDOMAIN
-        )
-        self._network_data = await self._wiser_rest_controller._get_hub_data(
-            WISERHUBNETWORK
-        )
-        self._schedule_data = await self._wiser_rest_controller._get_hub_data(
-            WISERHUBSCHEDULES
-        )
+    async def _get_hub_data(self) -> bool:
         try:
-            self._status_data = (
-                await self._wiser_rest_controller._get_hub_data(WISERHUBSTATUS)
+            start_time = datetime.now()
+            self._domain_data = await self._wiser_rest_controller.get_hub_data(
+                WISERHUBDOMAIN
             )
-        except WiserHubRESTError:
-            self._status_data = {}
-
-        # Set hub name on rest controller
-        self._wiser_rest_controller._hub_name = (
-            self._network_data.get("Station", {})
-            .get("NetworkInterface", {})
-            .get("HostName", "")
-        )
-
-        # load extra data
-        self._wiser_rest_controller._extra_config_file = (
-            self._extra_config_file
-        )
-        await self._wiser_rest_controller._get_extra_config_data()
+            self._network_data = (
+                await self._wiser_rest_controller.get_hub_data(WISERHUBNETWORK)
+            )
+            self._schedule_data = (
+                await self._wiser_rest_controller.get_hub_data(
+                    WISERHUBSCHEDULES
+                )
+            )
+            try:
+                self._status_data = (
+                    await self._wiser_rest_controller.get_hub_data(
+                        WISERHUBSTATUS
+                    )
+                )
+            except WiserHubRESTError:
+                self._status_data = {}
 
-        # Only get opentherm data if connected
-        if (
-            self._domain_data.get("System", {}).get(
+            # Only get opentherm data if connected
+            opentherm = self._domain_data.get("System", {}).get(
                 "OpenThermConnectionStatus", ""
             )
-            == "Connected"
-        ):
-            self._opentherm_data = (
-                await self._wiser_rest_controller._get_hub_data(
-                    WISERHUBOPENTHERM, False
+            if opentherm and opentherm == "Connected":
+                self._opentherm_data = (
+                    await self._wiser_rest_controller.get_hub_data(
+                        WISERHUBOPENTHERM, False
+                    )
                 )
+        except (
+            WiserHubConnectionError,
+            WiserHubAuthenticationError,
+            WiserHubRESTError,
+        ) as ex:
+            _LOGGER.debug("Update from Wiser hub failed. %s", ex)
+            raise ex
+        else:
+            # Set hub name on rest controller
+            self._wiser_rest_controller._hub_name = (
+                self._network_data.get("Station", {})
+                .get("NetworkInterface", {})
+                .get("HostName", "")
             )
 
-        if self._domain_data != {} and self._network_data != {}:
-            # System Object
-            _device_data = self._domain_data.get("Device", [])
-            self._system = _WiserSystem(
-                self._wiser_rest_controller,
-                self._domain_data,
-                self._network_data,
-                _device_data,
-                self._opentherm_data,
+            _LOGGER.debug(
+                "Update from %s successful and took %ss",
+                self._wiser_rest_controller._hub_name,
+                (datetime.now() - start_time).total_seconds(),
             )
+            return True
 
-            # Schedules Collection
-            self._schedules = _WiserScheduleCollection(
-                self._wiser_rest_controller,
-                self._schedule_data,
-                self._system.sunrise_times,
-                self._system.sunset_times,
-            )
+    async def _build_objects(self):
+        """Read all data from hub and populate objects"""
 
-            # Devices Collection
-            self._devices = _WiserDeviceCollection(
-                self._wiser_rest_controller, self._domain_data, self._schedules
-            )
+        # Read data from hub
+        try:
+            await self._get_hub_data()
 
-            # Rooms Collection
-            room_data = self._domain_data.get("Room", [])
-            self._rooms = _WiserRoomCollection(
-                self._wiser_rest_controller,
-                room_data,
-                self._schedules.get_by_type(WiserScheduleTypeEnum.heating),
-                self._devices,
-                self._enable_automations,
+            # load extra data
+            self._wiser_rest_controller._extra_config_file = (
+                self._extra_config_file
             )
+            await self._wiser_rest_controller.get_extra_config_data()
 
-            # Hot Water
-            if self._domain_data.get("HotWater"):
-                schedule = self._schedules.get_by_id(
-                    WiserScheduleTypeEnum.onoff,
-                    self._domain_data.get("HotWater")[0].get("ScheduleId", 0),
+            if self._domain_data != {} and self._network_data != {}:
+                # System Object
+                _device_data = self._domain_data.get("Device", [])
+                self._system = _WiserSystem(
+                    self._wiser_rest_controller,
+                    self._domain_data,
+                    self._network_data,
+                    _device_data,
+                    self._opentherm_data,
                 )
-                self._hotwater = _WiserHotwater(
+
+                # Schedules Collection
+                self._schedules = _WiserScheduleCollection(
                     self._wiser_rest_controller,
-                    self._domain_data.get("HotWater", {})[0],
-                    schedule,
+                    self._schedule_data,
+                    self._system.sunrise_times,
+                    self._system.sunset_times,
                 )
 
-            # Heating Channels
-            if self._domain_data.get("HeatingChannel"):
-                self._heating_channels = _WiserHeatingChannelCollection(
-                    self._domain_data.get("HeatingChannel"), self._rooms
+                # Devices Collection
+                self._devices = _WiserDeviceCollection(
+                    self._wiser_rest_controller,
+                    self._domain_data,
+                    self._schedules,
                 )
 
-            # Moments
-            if self._domain_data.get("Moment"):
-                self._moments = _WiserMomentCollection(
+                # Rooms Collection
+                room_data = self._domain_data.get("Room", [])
+                self._rooms = _WiserRoomCollection(
                     self._wiser_rest_controller,
-                    self._domain_data.get("Moment"),
+                    room_data,
+                    self._schedules.get_by_type(WiserScheduleTypeEnum.heating),
+                    self._devices,
+                    self._enable_automations,
                 )
 
-            # If gets here with no exceptions then success and return true
-            return True
+                # Hot Water
+                if self._domain_data.get("HotWater"):
+                    schedule = self._schedules.get_by_id(
+                        WiserScheduleTypeEnum.onoff,
+                        self._domain_data.get("HotWater")[0].get(
+                            "ScheduleId", 0
+                        ),
+                    )
+                    self._hotwater = _WiserHotwater(
+                        self._wiser_rest_controller,
+                        self._domain_data.get("HotWater", {})[0],
+                        schedule,
+                    )
+
+                # Heating Channels
+                if self._domain_data.get("HeatingChannel"):
+                    self._heating_channels = _WiserHeatingChannelCollection(
+                        self._domain_data.get("HeatingChannel"), self._rooms
+                    )
+
+                # Moments
+                if self._domain_data.get("Moment"):
+                    self._moments = _WiserMomentCollection(
+                        self._wiser_rest_controller,
+                        self._domain_data.get("Moment"),
+                    )
 
-        return False
+                # If gets here with no exceptions then success and return true
+                return True
+        except (
+            WiserHubConnectionError,
+            WiserHubAuthenticationError,
+            WiserHubRESTError,
+        ) as ex:
+            raise ex
 
     # API properties
     @property
     def api_parameters(self):
+        """Rest control api parameters."""
         return self._wiser_rest_controller._api_parameters
 
     @property
@@ -327,7 +356,7 @@ class WiserAPI:
 
         # Get raw json data
         if endpoint:
-            data = self._wiser_rest_controller._get_hub_data(endpoint)
+            data = self._wiser_rest_controller.get_hub_data(endpoint)
             try:
                 if data:
                     # Write out to file

{aioWiserHeatAPI-1.5.10 → aioWiserHeatAPI-1.5.11}/aioWiserHeatAPI.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: aioWiserHeatAPI
-Version: 1.5.10
+Version: 1.5.11
 Summary: An AsyncIO API for controlling the Drayton Wiser Heating system
 Home-page: https://github.com/msp1974/aioWiserHeatAPI
 Author: Mark Parker
@@ -18,7 +18,7 @@ Requires-Dist: aiohttp>=3.9.3
 Requires-Dist: aiofiles>=23.2.1
 Requires-Dist: pyyaml>=6.0.1
 
-# Drayton Wiser Hub API Async v1.5.10
+# Drayton Wiser Hub API Async v1.5.11
 
 This repository contains a simple API which queries the Drayton Wiser Heating sysystem used in the UK.
 
@@ -72,6 +72,11 @@ Documentation available in [info.md](https://github.com/msp1974/wiserHeatAPIv2/b
 
 ## Changelog
 
+### v1.5.11
+
+* Improve error management
+* Add additional debug logging
+
 ### v1.5.10
 
 * Add retry routine to prevent hub response errors