Started using flake8 to lint; removed some of its warnings.

[python_utils.git] / smart_home / registry.py

diff --git a/smart_home/registry.py b/smart_home/registry.py
index 20fb3f43807bf0dbbd45fa7f5c36ed8f4d1bc917..7349081f1b9b1637e2c18db7465b29e82626f54e 100644 (file)
--- a/smart_home/registry.py
+++ b/smart_home/registry.py
@@ -3,7 +3,6 @@
 import logging
 import re
 from typing import List, Optional, Set
-import warnings
 
 import argparse_utils
 import config
@@ -64,9 +63,8 @@ class SmartHomeRegistry(object):
             try:
                 (mac, name, keywords) = line.split(",")
             except ValueError:
-                msg = f'SH-CONFIG> {line} is malformed?!'
+                msg = f'SH-CONFIG> "{line}" is malformed?!  Skipping it.'
                 logger.warning(msg)
-                warnings.warn(msg)
                 continue
             mac = mac.strip()
             name = name.strip()
@@ -187,10 +185,12 @@ class SmartHomeRegistry(object):
                     return device.Device(name, mac, kws)
             except Exception as e:
                 logger.exception(e)
+                logger.debug(
+                    f'Device {name} at {mac} with {kws} confused me, returning a generic Device'
+                )
                 return device.Device(name, mac, kws)
         msg = f'{mac} is not a known smart home device, returning None'
         logger.warning(msg)
-        warnings.warn(msg)
         return None
 
     def query(self, query: str) -> List[device.Device]: