From 3536622a2fcc555b4f15f0e86b8878cfba154db4 Mon Sep 17 00:00:00 2001 From: Bouwe Westerdijk Date: Tue, 25 Feb 2025 20:03:22 +0100 Subject: [PATCH 01/10] User proper super().__init__() --- plugwise/data.py | 2 +- plugwise/helper.py | 2 +- plugwise/legacy/data.py | 2 +- plugwise/legacy/helper.py | 2 +- plugwise/legacy/smile.py | 2 +- plugwise/smile.py | 2 +- 6 files changed, 6 insertions(+), 6 deletions(-) diff --git a/plugwise/data.py b/plugwise/data.py index b23b4e53a..baac1f815 100644 --- a/plugwise/data.py +++ b/plugwise/data.py @@ -27,9 +27,9 @@ class SmileData(SmileHelper): def __init__(self) -> None: """Init.""" + super().__init__() self._smile_props: SmileProps self._zones: dict[str, GwEntityData] = {} - SmileHelper.__init__(self) def _all_entity_data(self) -> None: """Helper-function for get_all_gateway_entities(). diff --git a/plugwise/helper.py b/plugwise/helper.py index 35472db5b..bcdd38d55 100644 --- a/plugwise/helper.py +++ b/plugwise/helper.py @@ -75,6 +75,7 @@ class SmileHelper(SmileCommon): def __init__(self) -> None: """Set the constructor for this class.""" + super().__init__() self._endpoint: str self._elga: bool self._is_thermostat: bool @@ -89,7 +90,6 @@ def __init__(self) -> None: self.smile_model: str self.smile_model_id: str | None self.smile_version: version.Version - SmileCommon.__init__(self) def _all_appliances(self) -> None: """Collect all appliances with relevant info. diff --git a/plugwise/legacy/data.py b/plugwise/legacy/data.py index 777f528a1..8a51f2992 100644 --- a/plugwise/legacy/data.py +++ b/plugwise/legacy/data.py @@ -17,8 +17,8 @@ class SmileLegacyData(SmileLegacyHelper): def __init__(self) -> None: """Init.""" + super().__init__() self._smile_props: SmileProps - SmileLegacyHelper.__init__(self) def _all_entity_data(self) -> None: """Helper-function for get_all_gateway_entities(). diff --git a/plugwise/legacy/helper.py b/plugwise/legacy/helper.py index 5d900aa20..84dc6d498 100644 --- a/plugwise/legacy/helper.py +++ b/plugwise/legacy/helper.py @@ -64,6 +64,7 @@ class SmileLegacyHelper(SmileCommon): def __init__(self) -> None: """Set the constructor for this class.""" + super().__init__() self._appliances: etree.Element self._is_thermostat: bool self._loc_data: dict[str, ThermoLoc] @@ -75,7 +76,6 @@ def __init__(self) -> None: self.smile_model: str self.smile_version: Version self.smile_zigbee_mac_address: str | None - SmileCommon.__init__(self) def _all_appliances(self) -> None: """Collect all appliances with relevant info.""" diff --git a/plugwise/legacy/smile.py b/plugwise/legacy/smile.py index 2d1e42eef..c9c3121c6 100644 --- a/plugwise/legacy/smile.py +++ b/plugwise/legacy/smile.py @@ -54,6 +54,7 @@ def __init__( smile_zigbee_mac_address: str | None, ) -> None: """Set the constructor for this class.""" + super().__init__() self._cooling_present = False self._is_thermostat = _is_thermostat self._loc_data = _loc_data @@ -71,7 +72,6 @@ def __init__( self.smile_type = smile_type self.smile_version = smile_version self.smile_zigbee_mac_address = smile_zigbee_mac_address - SmileLegacyData.__init__(self) self._first_update = True self._previous_day_number: str = "0" diff --git a/plugwise/smile.py b/plugwise/smile.py index 7fd951d71..7048ee275 100644 --- a/plugwise/smile.py +++ b/plugwise/smile.py @@ -62,6 +62,7 @@ def __init__( smile_version: Version, ) -> None: """Set the constructor for this class.""" + super().__init__() self._cooling_present = _cooling_present self._elga = _elga self._is_thermostat = _is_thermostat @@ -81,7 +82,6 @@ def __init__( self.smile_type = smile_type self.smile_version = smile_version self.therms_with_offset_func: list[str] = [] - SmileData.__init__(self) async def full_xml_update(self) -> None: """Perform a first fetch of the Plugwise server XML data.""" From 42bcfe83f09167156fd0dc1b5ce709e992b2beb0 Mon Sep 17 00:00:00 2001 From: Bouwe Westerdijk Date: Wed, 26 Feb 2025 18:04:08 +0100 Subject: [PATCH 02/10] Add SmileCommon property heater_id, Smile/Legacy property cooling_present --- plugwise/__init__.py | 8 ++------ plugwise/common.py | 5 +++++ plugwise/helper.py | 6 +++--- plugwise/legacy/helper.py | 4 ++-- plugwise/legacy/smile.py | 5 +++++ plugwise/smile.py | 5 +++++ 6 files changed, 22 insertions(+), 11 deletions(-) diff --git a/plugwise/__init__.py b/plugwise/__init__.py index 5674471e6..dd6c88a24 100644 --- a/plugwise/__init__.py +++ b/plugwise/__init__.py @@ -86,9 +86,7 @@ def __init__( @property def cooling_present(self) -> bool: """Return the cooling capability.""" - if "cooling_present" in self._smile_props: - return self._smile_props["cooling_present"] - return False + return self._smile_api.cooling_present @property def gateway_id(self) -> str: @@ -98,9 +96,7 @@ def gateway_id(self) -> str: @property def heater_id(self) -> str: """Return the heater-id.""" - if "heater_id" in self._smile_props: - return self._smile_props["heater_id"] - return NONE + return self._smile_api.heater_id @property def item_count(self) -> int: diff --git a/plugwise/common.py b/plugwise/common.py index 03ec7c865..c261eeb4b 100644 --- a/plugwise/common.py +++ b/plugwise/common.py @@ -58,6 +58,11 @@ def __init__(self) -> None: self.smile_name: str self.smile_type: str + @property + def heater_id(self) -> str: + """Return the heater-id.""" + return self._heater_id + def smile(self, name: str) -> bool: """Helper-function checking the smile-name.""" return self.smile_name == name diff --git a/plugwise/helper.py b/plugwise/helper.py index bcdd38d55..13d487a9e 100644 --- a/plugwise/helper.py +++ b/plugwise/helper.py @@ -233,7 +233,7 @@ def _appliance_info_finder(self, appl: Munch, appliance: etree.Element) -> Munch appliance, "domestic_hot_water_mode_control_functionality" ) # Skip orphaned heater_central (Core Issue #104433) - if appl.entity_id != self._heater_id: + if appl.entity_id != self.heater_id: return Munch() return appl case _ as s if s.endswith("_plug"): @@ -344,7 +344,7 @@ def _get_measurement_data(self, entity_id: str) -> GwEntityData: # Get non-P1 data from APPLIANCES measurements = DEVICE_MEASUREMENTS - if self._is_thermostat and entity_id == self._heater_id: + if self._is_thermostat and entity_id == self.heater_id: measurements = HEATER_CENTRAL_MEASUREMENTS # Show the allowed dhw_modes (Loria only) if self._dhw_allowed_modes: @@ -619,7 +619,7 @@ def _update_anna_cooling(self, entity_id: str, data: GwEntityData) -> None: Support added for Techneco Elga and Thercon Loria/Thermastage. """ - if entity_id != self._heater_id: + if entity_id != self.heater_id: return if "elga_status_code" in data: diff --git a/plugwise/legacy/helper.py b/plugwise/legacy/helper.py index 84dc6d498..6c512b1a9 100644 --- a/plugwise/legacy/helper.py +++ b/plugwise/legacy/helper.py @@ -125,7 +125,7 @@ def _all_appliances(self) -> None: continue # Skip orphaned heater_central (Core Issue #104433) - if appl.pwclass == "heater_central" and appl.entity_id != self._heater_id: + if appl.pwclass == "heater_central" and appl.entity_id != self.heater_id: continue # pragma: no cover self._create_gw_entities(appl) @@ -268,7 +268,7 @@ def _get_measurement_data(self, entity_id: str) -> GwEntityData: return data measurements = DEVICE_MEASUREMENTS - if self._is_thermostat and entity_id == self._heater_id: + if self._is_thermostat and entity_id == self.heater_id: measurements = HEATER_CENTRAL_MEASUREMENTS if ( diff --git a/plugwise/legacy/smile.py b/plugwise/legacy/smile.py index c9c3121c6..ef2d84bc2 100644 --- a/plugwise/legacy/smile.py +++ b/plugwise/legacy/smile.py @@ -76,6 +76,11 @@ def __init__( self._first_update = True self._previous_day_number: str = "0" + @property + def cooling_present(self) -> bool: + """Return the cooling capability.""" + return False + async def full_xml_update(self) -> None: """Perform a first fetch of the Plugwise server XML data.""" self._domain_objects = await self._request(DOMAIN_OBJECTS) diff --git a/plugwise/smile.py b/plugwise/smile.py index 7048ee275..6de0ebd06 100644 --- a/plugwise/smile.py +++ b/plugwise/smile.py @@ -83,6 +83,11 @@ def __init__( self.smile_version = smile_version self.therms_with_offset_func: list[str] = [] + @property + def cooling_present(self) -> bool: + """Return the cooling capability.""" + return self._cooling_present + async def full_xml_update(self) -> None: """Perform a first fetch of the Plugwise server XML data.""" self._domain_objects = await self._request(DOMAIN_OBJECTS) From 40f60ad9a37136bb79219c40c9e1715d081eb7a5 Mon Sep 17 00:00:00 2001 From: Bouwe Westerdijk Date: Wed, 26 Feb 2025 18:07:47 +0100 Subject: [PATCH 03/10] Add Smile/Legacy property gateway_id --- plugwise/__init__.py | 2 +- plugwise/helper.py | 5 +++++ plugwise/legacy/helper.py | 16 +++++++++++----- 3 files changed, 17 insertions(+), 6 deletions(-) diff --git a/plugwise/__init__.py b/plugwise/__init__.py index dd6c88a24..8ff725add 100644 --- a/plugwise/__init__.py +++ b/plugwise/__init__.py @@ -91,7 +91,7 @@ def cooling_present(self) -> bool: @property def gateway_id(self) -> str: """Return the gateway-id.""" - return self._smile_props["gateway_id"] + return self._smile_api.gateway_id @property def heater_id(self) -> str: diff --git a/plugwise/helper.py b/plugwise/helper.py index 13d487a9e..90ac2ba6c 100644 --- a/plugwise/helper.py +++ b/plugwise/helper.py @@ -91,6 +91,11 @@ def __init__(self) -> None: self.smile_model_id: str | None self.smile_version: version.Version + @property + def gateway_id(self) -> str: + """Return the gateway-id.""" + return self._gateway_id + def _all_appliances(self) -> None: """Collect all appliances with relevant info. diff --git a/plugwise/legacy/helper.py b/plugwise/legacy/helper.py index 6c512b1a9..a620dd0f4 100644 --- a/plugwise/legacy/helper.py +++ b/plugwise/legacy/helper.py @@ -66,6 +66,7 @@ def __init__(self) -> None: """Set the constructor for this class.""" super().__init__() self._appliances: etree.Element + self._gateway_id: str = NONE self._is_thermostat: bool self._loc_data: dict[str, ThermoLoc] self._locations: etree.Element @@ -77,6 +78,11 @@ def __init__(self) -> None: self.smile_version: Version self.smile_zigbee_mac_address: str | None + @property + def gateway_id(self) -> str: + """Return the gateway-id.""" + return self._gateway_id + def _all_appliances(self) -> None: """Collect all appliances with relevant info.""" self._count = 0 @@ -173,11 +179,11 @@ def _create_legacy_gateway(self) -> None: Use the home_location or FAKE_APPL as entity id. """ - self.gateway_id = self._home_loc_id + self._gateway_id = self._home_loc_id if self.smile_type == "power": - self.gateway_id = FAKE_APPL + self._gateway_id = FAKE_APPL - self.gw_entities[self.gateway_id] = {"dev_class": "gateway"} + self.gw_entities[self._gateway_id] = {"dev_class": "gateway"} self._count += 1 for key, value in { "firmware": str(self.smile_version), @@ -190,7 +196,7 @@ def _create_legacy_gateway(self) -> None: }.items(): if value is not None: gw_key = cast(ApplianceType, key) - self.gw_entities[self.gateway_id][gw_key] = value + self.gw_entities[self._gateway_id][gw_key] = value self._count += 1 def _appliance_info_finder(self, appliance: etree, appl: Munch) -> Munch: @@ -282,7 +288,7 @@ def _get_measurement_data(self, entity_id: str) -> GwEntityData: # Anna: the Smile outdoor_temperature is present in the Home location # For some Anna's LOCATIONS is empty, falling back to domain_objects! - if self._is_thermostat and entity_id == self.gateway_id: + if self._is_thermostat and entity_id == self._gateway_id: locator = f"./location[@id='{self._home_loc_id}']/logs/point_log[type='outdoor_temperature']/period/measurement" if (found := self._domain_objects.find(locator)) is not None: value = format_measure(found.text, NONE) From d665c4517a3bff835acc848af566a47b3da48ea9 Mon Sep 17 00:00:00 2001 From: Bouwe Westerdijk Date: Wed, 26 Feb 2025 19:01:55 +0100 Subject: [PATCH 04/10] Add Smile/Legacy item_count property --- plugwise/__init__.py | 2 +- plugwise/helper.py | 5 +++++ plugwise/legacy/helper.py | 5 +++++ 3 files changed, 11 insertions(+), 1 deletion(-) diff --git a/plugwise/__init__.py b/plugwise/__init__.py index 8ff725add..8363b9874 100644 --- a/plugwise/__init__.py +++ b/plugwise/__init__.py @@ -101,7 +101,7 @@ def heater_id(self) -> str: @property def item_count(self) -> int: """Return the item-count.""" - return self._smile_props["item_count"] + return self._smile_api.item_count @property def reboot(self) -> bool: diff --git a/plugwise/helper.py b/plugwise/helper.py index 90ac2ba6c..87c4e986e 100644 --- a/plugwise/helper.py +++ b/plugwise/helper.py @@ -96,6 +96,11 @@ def gateway_id(self) -> str: """Return the gateway-id.""" return self._gateway_id + @property + def item_count(self) -> int: + """Return the item-count.""" + return self._count + def _all_appliances(self) -> None: """Collect all appliances with relevant info. diff --git a/plugwise/legacy/helper.py b/plugwise/legacy/helper.py index a620dd0f4..e22b489e5 100644 --- a/plugwise/legacy/helper.py +++ b/plugwise/legacy/helper.py @@ -83,6 +83,11 @@ def gateway_id(self) -> str: """Return the gateway-id.""" return self._gateway_id + @property + def item_count(self) -> int: + """Return the item-count.""" + return self._count + def _all_appliances(self) -> None: """Collect all appliances with relevant info.""" self._count = 0 From febd1e4d6d346750654b04fc6614cb4fe2e3e3bd Mon Sep 17 00:00:00 2001 From: Bouwe Westerdijk Date: Wed, 26 Feb 2025 19:08:59 +0100 Subject: [PATCH 05/10] Cleaning up SmileProps --- plugwise/__init__.py | 4 ---- plugwise/constants.py | 11 ----------- plugwise/data.py | 12 +----------- plugwise/legacy/data.py | 10 ++-------- plugwise/legacy/smile.py | 3 --- plugwise/smile.py | 11 ++++------- 6 files changed, 7 insertions(+), 44 deletions(-) diff --git a/plugwise/__init__.py b/plugwise/__init__.py index 8363b9874..35250323d 100644 --- a/plugwise/__init__.py +++ b/plugwise/__init__.py @@ -18,7 +18,6 @@ STATUS, SYSTEM, GwEntityData, - SmileProps, ThermoLoc, ) from plugwise.exceptions import ( @@ -69,7 +68,6 @@ def __init__( self._opentherm_device = False self._schedule_old_states: dict[str, dict[str, str]] = {} self._smile_api: SmileAPI | SmileLegacyAPI - self._smile_props: SmileProps = {} self._stretch_v2 = False self._target_smile: str = NONE self.smile_hostname: str = NONE @@ -158,7 +156,6 @@ async def connect(self) -> Version: self._opentherm_device, self._request, self._schedule_old_states, - self._smile_props, self.smile_hostname, self.smile_hw_version, self.smile_mac_address, @@ -175,7 +172,6 @@ async def connect(self) -> Version: self._on_off_device, self._opentherm_device, self._request, - self._smile_props, self._stretch_v2, self._target_smile, self.smile_hostname, diff --git a/plugwise/constants.py b/plugwise/constants.py index 7d406ad94..f12c8bbd9 100644 --- a/plugwise/constants.py +++ b/plugwise/constants.py @@ -393,17 +393,6 @@ ) -class SmileProps(TypedDict, total=False): - """The SmileProps Data class.""" - - cooling_present: bool - gateway_id: str - heater_id: str - item_count: int - reboot: bool - smile_name: str - - class ModuleData(TypedDict): """The Module data class.""" diff --git a/plugwise/data.py b/plugwise/data.py index baac1f815..97ec5b51a 100644 --- a/plugwise/data.py +++ b/plugwise/data.py @@ -16,7 +16,6 @@ OFF, ActuatorData, GwEntityData, - SmileProps, ) from plugwise.helper import SmileHelper from plugwise.util import remove_empty_platform_dicts @@ -28,27 +27,18 @@ class SmileData(SmileHelper): def __init__(self) -> None: """Init.""" super().__init__() - self._smile_props: SmileProps self._zones: dict[str, GwEntityData] = {} def _all_entity_data(self) -> None: """Helper-function for get_all_gateway_entities(). - Collect data for each entity and add to self._smile_props and self.gw_entities. + Collect data for each entity and add to self.gw_entities. """ self._update_gw_entities() if self.smile(ADAM): self._update_zones() self.gw_entities.update(self._zones) - self._smile_props["gateway_id"] = self._gateway_id - self._smile_props["item_count"] = self._count - self._smile_props["reboot"] = True - self._smile_props["smile_name"] = self.smile_name - if self._is_thermostat: - self._smile_props["heater_id"] = self._heater_id - self._smile_props["cooling_present"] = self._cooling_present - def _update_zones(self) -> None: """Helper-function for _all_entity_data() and async_update(). diff --git a/plugwise/legacy/data.py b/plugwise/legacy/data.py index 8a51f2992..debd460fc 100644 --- a/plugwise/legacy/data.py +++ b/plugwise/legacy/data.py @@ -7,7 +7,7 @@ # Dict as class # Version detection -from plugwise.constants import NONE, OFF, GwEntityData, SmileProps +from plugwise.constants import NONE, OFF, GwEntityData from plugwise.legacy.helper import SmileLegacyHelper from plugwise.util import remove_empty_platform_dicts @@ -18,19 +18,13 @@ class SmileLegacyData(SmileLegacyHelper): def __init__(self) -> None: """Init.""" super().__init__() - self._smile_props: SmileProps def _all_entity_data(self) -> None: """Helper-function for get_all_gateway_entities(). - Collect data for each entity and add to self._smile_props and self.gw_entities. + Collect data for each entity and add to self.gw_entities. """ self._update_gw_entities() - self._smile_props["gateway_id"] = self.gateway_id - self._smile_props["item_count"] = self._count - self._smile_props["smile_name"] = self.smile_name - if self._is_thermostat: - self._smile_props["heater_id"] = self._heater_id def _update_gw_entities(self) -> None: """Helper-function for _all_entity_data() and async_update(). diff --git a/plugwise/legacy/smile.py b/plugwise/legacy/smile.py index ef2d84bc2..eedd360f7 100644 --- a/plugwise/legacy/smile.py +++ b/plugwise/legacy/smile.py @@ -19,7 +19,6 @@ REQUIRE_APPLIANCES, RULES, GwEntityData, - SmileProps, ThermoLoc, ) from plugwise.exceptions import ConnectionFailedError, DataMissingError, PlugwiseError @@ -41,7 +40,6 @@ def __init__( _on_off_device: bool, _opentherm_device: bool, _request: Callable[..., Awaitable[Any]], - _smile_props: SmileProps, _stretch_v2: bool, _target_smile: str, smile_hostname: str, @@ -61,7 +59,6 @@ def __init__( self._on_off_device = _on_off_device self._opentherm_device = _opentherm_device self._request = _request - self._smile_props = _smile_props self._stretch_v2 = _stretch_v2 self._target_smile = _target_smile self.smile_hostname = smile_hostname diff --git a/plugwise/smile.py b/plugwise/smile.py index 6de0ebd06..ee3a19e65 100644 --- a/plugwise/smile.py +++ b/plugwise/smile.py @@ -22,7 +22,6 @@ OFF, RULES, GwEntityData, - SmileProps, ThermoLoc, ) from plugwise.data import SmileData @@ -51,7 +50,6 @@ def __init__( _opentherm_device: bool, _request: Callable[..., Awaitable[Any]], _schedule_old_states: dict[str, dict[str, str]], - _smile_props: SmileProps, smile_hostname: str | None, smile_hw_version: str | None, smile_mac_address: str | None, @@ -72,7 +70,6 @@ def __init__( self._opentherm_device = _opentherm_device self._request = _request self._schedule_old_states = _schedule_old_states - self._smile_props = _smile_props self.smile_hostname = smile_hostname self.smile_hw_version = smile_hw_version self.smile_mac_address = smile_mac_address @@ -126,8 +123,8 @@ async def async_update(self) -> dict[str, GwEntityData]: self.get_all_gateway_entities() # Set self._cooling_enabled - required for set_temperature(), # also, check for a failed data-retrieval - if "heater_id" in self._smile_props: - heat_cooler = self.gw_entities[self._smile_props["heater_id"]] + if self.heater_id != NONE: + heat_cooler = self.gw_entities[self.heater_id] if ( "binary_sensors" in heat_cooler and "cooling_enabled" in heat_cooler["binary_sensors"] @@ -136,7 +133,7 @@ async def async_update(self) -> dict[str, GwEntityData]: "cooling_enabled" ] else: # cover failed data-retrieval for P1 - _ = self.gw_entities[self._smile_props["gateway_id"]]["location"] + _ = self.gw_entities[self.gateway_id]["location"] except KeyError as err: raise DataMissingError("No Plugwise actual data received") from err @@ -278,7 +275,7 @@ async def set_gateway_mode(self, mode: str) -> None: f"{valid}" "" ) - uri = f"{APPLIANCES};id={self._smile_props['gateway_id']}/gateway_mode_control" + uri = f"{APPLIANCES};id={self.gateway_id}/gateway_mode_control" await self.call_request(uri, method="put", data=data) async def set_regulation_mode(self, mode: str) -> None: From 2473a8b3a1b23a9d4253791f68447c309de03e16 Mon Sep 17 00:00:00 2001 From: Bouwe Westerdijk Date: Wed, 26 Feb 2025 19:09:58 +0100 Subject: [PATCH 06/10] Import NONE --- plugwise/smile.py | 1 + 1 file changed, 1 insertion(+) diff --git a/plugwise/smile.py b/plugwise/smile.py index ee3a19e65..f3b259ff8 100644 --- a/plugwise/smile.py +++ b/plugwise/smile.py @@ -18,6 +18,7 @@ LOCATIONS, MAX_SETPOINT, MIN_SETPOINT, + NONE, NOTIFICATIONS, OFF, RULES, From 608c555c0653a5a5749bc5e7708f8ed287f635bb Mon Sep 17 00:00:00 2001 From: Bouwe Westerdijk Date: Wed, 26 Feb 2025 19:16:24 +0100 Subject: [PATCH 07/10] Update CHANGELOG --- CHANGELOG.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 0cba877ea..2dd6a39b2 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -3,7 +3,7 @@ ## Ongoing - Improve readability of xml-data in POST/PUT requests via [#707](https://github.com/plugwise/python-plugwise/pull/707), [#708](https://github.com/plugwise/python-plugwise/pull/708) and [#715](https://github.com/plugwise/python-plugwise/pull/715) -- Continuous improvements via [#711](https://github.com/plugwise/python-plugwise/pull/711) and [#713](https://github.com/plugwise/python-plugwise/pull/713) +- Continuous improvements via [#711](https://github.com/plugwise/python-plugwise/pull/711), [#713](https://github.com/plugwise/python-plugwise/pull/713) and [#716](https://github.com/plugwise/python-plugwise/pull/716) ## v1.7.2 From c8dec0a6b2b2c492617aafb48c6dfd7795a7fa75 Mon Sep 17 00:00:00 2001 From: Bouwe Westerdijk Date: Wed, 26 Feb 2025 19:32:21 +0100 Subject: [PATCH 08/10] Remove pylint_per-file-ignores plugin --- pyproject.toml | 7 ------- 1 file changed, 7 deletions(-) diff --git a/pyproject.toml b/pyproject.toml index e5ca6ede0..d46e0806b 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -75,7 +75,6 @@ init-hook = """\ load-plugins = [ "pylint.extensions.code_style", "pylint.extensions.typing", - "pylint_per_file_ignores", ] persistent = false extension-pkg-allow-list = [ @@ -366,12 +365,6 @@ enable = [ #"useless-suppression", # temporarily every now and then to clean them up "use-symbolic-message-instead", ] -per-file-ignores = [ - # redefined-outer-name: Tests reference fixtures in the test function - # use-implicit-booleaness-not-comparison: Tests need to validate that a list - # or a dict is returned - "/tests/:redefined-outer-name,use-implicit-booleaness-not-comparison", -] [tool.pylint.REPORTS] score = false From d335cfc08a82fa2e33128b4b4470d021ce5b3e2b Mon Sep 17 00:00:00 2001 From: Bouwe Westerdijk Date: Wed, 26 Feb 2025 19:40:32 +0100 Subject: [PATCH 09/10] Remove init in legacy/data.py --- plugwise/legacy/data.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/plugwise/legacy/data.py b/plugwise/legacy/data.py index debd460fc..03ee4a014 100644 --- a/plugwise/legacy/data.py +++ b/plugwise/legacy/data.py @@ -15,10 +15,6 @@ class SmileLegacyData(SmileLegacyHelper): """The Plugwise Smile main class.""" - def __init__(self) -> None: - """Init.""" - super().__init__() - def _all_entity_data(self) -> None: """Helper-function for get_all_gateway_entities(). From 5803c172f0b5f1a660ef5a80cef5456fbc032443 Mon Sep 17 00:00:00 2001 From: Bouwe Westerdijk Date: Wed, 26 Feb 2025 19:53:56 +0100 Subject: [PATCH 10/10] Bump to v1.7.3a3 --- pyproject.toml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pyproject.toml b/pyproject.toml index d46e0806b..4b8283c23 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta" [project] name = "plugwise" -version = "1.7.3a2" +version = "1.7.3a3" license = {file = "LICENSE"} description = "Plugwise Smile (Adam/Anna/P1) and Stretch module for Python 3." readme = "README.md"