Merge pull request #17 from dala318/log_levels

Log levels

Author: dala318

__init__.py

@@ -23,7 +23,7 @@
 _LOGGER = logging.getLogger(__name__)
 
 UPDATE_TOPIC = f"{DOMAIN}_update"
-ERROR_ITERVAL_MAPPING = [10, 60, 300, 600, 3000, 6000]
+ERROR_ITERVAL_MAPPING = [0, 10, 60, 300, 600, 3000, 6000]
 NOTIFICATION_ID = "ph803w_device_notification"
 NOTIFICATION_TITLE = "PH-803W Device status"
 

lib/device.py

@@ -91,6 +91,7 @@ def _run(self, once: bool = True) -> bool:
         # from now on some cyclig bahavior
         data = bytes.fromhex("000000030400009002")
         self._socket.sendall(data)
+        self._empty_counter = 0
 
         # If continous reading ping/pong needs to be run cyclic
         if not once:
@@ -103,7 +104,7 @@ def _run(self, once: bool = True) -> bool:
         while self._loop:
             response = self._socket.recv(1024)
             if self._empty_counter > ABORT_AFTER_CONSECUTIVE_EMPTY:
-                _LOGGER.error("Too many empty consecutive packages")
+                _LOGGER.warning("Too many empty consecutive packages")
                 raise DeviceError("Too many empty consecutive packages")
             if len(response) == 0:
                 self._empty_counter += 1
@@ -267,7 +268,7 @@ def get(self) -> float:
                     return val
         except StatisticsError:
             return self._values[-1]
-        _LOGGER.error("No match in outlier filter shall never happen!")
+        _LOGGER.warning("No match in outlier filter shall never happen!")
         return self._values[-1]
 
 

lib/discovery.py

@@ -37,20 +37,24 @@ def run(self):
         # Receive device reaponse to discovery
         data, remote = self._socket.recvfrom(1024)
         if data[0] != 0 and data[1] != 0 and data[2] != 0 and data[2] != 3:
-            _LOGGER.error("Ignore data package because invalid prefix: %s" % data[0:3])
+            _LOGGER.warning(
+                "Ignore data package because invalid prefix: %s" % data[0:3]
+            )
             raise DiscoveryError("Ignore data package because invalid prefix")
         data_length = data[4]
         if len(data) != data_length + 5:
-            _LOGGER.error(
+            _LOGGER.warning(
                 "Ignore data package because invalid length(%s): %s"
                 % (data_length, data)
             )
             raise DiscoveryError("Ignore data package because invalid length")
         if data[7] == 3:
-            _LOGGER.error("Unknown response message type")
+            _LOGGER.warning("Unknown response message type")
             raise DiscoveryError("Unknown response message type")
         if data[7] != 4:
-            _LOGGER.error("Ignore data package because invalid message type ${data[7]}")
+            _LOGGER.warning(
+                "Ignore data package because invalid message type ${data[7]}"
+            )
             raise DiscoveryError("Ignore data package because invalid message type")
 
         # Parsing result of correct type