Cleanup, improve docstring

Author: bouwew

plugwise/common.py

@@ -9,9 +9,7 @@
 
 from plugwise.constants import (
     ANNA,
-    GROUP_MEASUREMENTS,
     GROUP_TYPES,
-    LOGGER,
     NONE,
     PRIORITY_DEVICE_CLASSES,
     SPECIAL_PLUG_TYPES,
@@ -23,8 +21,6 @@
 from plugwise.util import (
     check_heater_central,
     check_model,
-    common_match_cases,
-    format_measure,
     get_vendor_name,
     return_valid,
 )
@@ -195,15 +191,14 @@ def _entity_switching_group(self, entity: GwEntityData, data: GwEntityData) -> N
     def _get_groups(self) -> dict[str, GwEntityData]:
         """Helper-function for smile.py: get_all_gateway_entities().
 
-        Collect switching- or pump-group info.
+        Collect switching-, pumping- or report-group info.
         """
         groups: dict[str, GwEntityData] = {}
         # P1 and Anna don't have groups
         if self.smile.type == "power" or self.check_name(ANNA):
             return groups
 
         for group in self._domain_objects.findall("./group"):
-            group_sensors = {}
             members: list[str] = []
             group_id = group.attrib["id"]
             group_name = group.find("name").text

plugwise/helper.py

@@ -328,7 +328,6 @@ def _get_measurement_data(self, entity_id: str) -> GwEntityData:
         """
         data: GwEntityData = {"binary_sensors": {}, "sensors": {}, "switches": {}}
         entity = self.gw_entities[entity_id]
-        LOGGER.debug("HOI entity: %s", entity)
 
         # Get P1 smartmeter data from LOCATIONS
         smile_is_power = self.smile.type == "power"
@@ -385,7 +384,7 @@ def _collect_group_sensors(
     ) -> None:
         """Collect group sensors."""
         if (group := self._domain_objects.find(f'./group[@id="{entity_id}"]')) is None:
-            return None
+            return
 
         for measurement, attrs in measurements.items():
             locator = f'.//logs/point_log[type="{measurement}"]/period/measurement'