PySwitchbot 0.60.1__tar.gz → 0.62.0__tar.gz

{pyswitchbot-0.60.1 → pyswitchbot-0.62.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: PySwitchbot
-Version: 0.60.1
+Version: 0.62.0
 Summary: A library to communicate with Switchbot
 Home-page: https://github.com/sblibs/pySwitchbot/
 Author: Daniel Hjelseth Hoyer

{pyswitchbot-0.60.1 → pyswitchbot-0.62.0}/PySwitchbot.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: PySwitchbot
-Version: 0.60.1
+Version: 0.62.0
 Summary: A library to communicate with Switchbot
 Home-page: https://github.com/sblibs/pySwitchbot/
 Author: Daniel Hjelseth Hoyer

{pyswitchbot-0.60.1 → pyswitchbot-0.62.0}/PySwitchbot.egg-info/SOURCES.txt

@@ -16,6 +16,7 @@ switchbot/enum.py
 switchbot/helpers.py
 switchbot/models.py
 switchbot/adv_parsers/__init__.py
+switchbot/adv_parsers/air_purifier.py
 switchbot/adv_parsers/blind_tilt.py
 switchbot/adv_parsers/bot.py
 switchbot/adv_parsers/bulb.py
@@ -36,12 +37,15 @@ switchbot/adv_parsers/plug.py
 switchbot/adv_parsers/relay_switch.py
 switchbot/adv_parsers/remote.py
 switchbot/adv_parsers/roller_shade.py
+switchbot/adv_parsers/vacuum.py
 switchbot/const/__init__.py
+switchbot/const/air_purifier.py
 switchbot/const/evaporative_humidifier.py
 switchbot/const/fan.py
 switchbot/const/hub2.py
 switchbot/const/lock.py
 switchbot/devices/__init__.py
+switchbot/devices/air_purifier.py
 switchbot/devices/base_cover.py
 switchbot/devices/base_light.py
 switchbot/devices/blind_tilt.py
@@ -62,7 +66,9 @@ switchbot/devices/motion.py
 switchbot/devices/plug.py
 switchbot/devices/relay_switch.py
 switchbot/devices/roller_shade.py
+switchbot/devices/vacuum.py
 tests/test_adv_parser.py
+tests/test_air_purifier.py
 tests/test_base_cover.py
 tests/test_blind_tilt.py
 tests/test_curtain.py
@@ -70,4 +76,5 @@ tests/test_evaporative_humidifier.py
 tests/test_fan.py
 tests/test_hub2.py
 tests/test_relay_switch.py
-tests/test_roller_shade.py
+tests/test_roller_shade.py
+tests/test_vacuum.py

{pyswitchbot-0.60.1 → pyswitchbot-0.62.0}/pyproject.toml

@@ -28,8 +28,13 @@ ignore = [
     "UP038", # Use `X | Y` in `isinstance` is slower
     "S603", #  check for execution of untrusted input
     "S105", # possible hard coded creds
+    "TID252", # not for this lib
+    "TRY003", # nice but too many to fix,
+    "G201", # too noisy
+    "PLR2004", # too many to fix
 ]
 select = [
+    "ASYNC", # async rules
     "B",   # flake8-bugbear
     "D",   # flake8-docstrings
     "C4",  # flake8-comprehensions
@@ -40,6 +45,24 @@ select = [
     "UP",  # pyupgrade
     "I",   # isort
     "RUF", # ruff specific
+    "FLY", # flynt
+    "G", # flake8-logging-format   ,
+    "PERF", # Perflint
+    "PGH", # pygrep-hooks
+    "PIE", # flake8-pie
+    "PL", # pylint
+    "PT", # flake8-pytest-style
+    "PTH", # flake8-pathlib
+    "PYI", # flake8-pyi
+    "RET", # flake8-return
+    "RSE", # flake8-raise    ,
+    "SIM", # flake8-simplify
+    "SLF", # flake8-self
+    "SLOT", # flake8-slots
+    "T100", # Trace found: {name} used
+    "T20", # flake8-print
+    "TID", # Tidy imports
+    "TRY", # tryceratops
 ]
 
 [tool.ruff.lint.per-file-ignores]
@@ -50,10 +73,15 @@ select = [
     "D103",
     "D104",
     "S101",
+    "SLF001",
+    "PLR2004",
 ]
 "setup.py" = ["D100"]
 "conftest.py" = ["D100"]
 "docs/conf.py" = ["D100"]
+"scripts/**/*" = [
+    "T201"
+]
 
 [tool.ruff.lint.isort]
 known-first-party = ["pySwitchbot", "tests"]

{pyswitchbot-0.60.1 → pyswitchbot-0.62.0}/setup.py

@@ -20,7 +20,7 @@ setup(
         "cryptography>=39.0.0",
         "pyOpenSSL>=23.0.0",
     ],
-    version="0.60.1",
+    version="0.62.0",
     description="A library to communicate with Switchbot",
     long_description=long_description,
     long_description_content_type="text/markdown",

{pyswitchbot-0.60.1 → pyswitchbot-0.62.0}/switchbot/__init__.py

@@ -10,6 +10,7 @@ from bleak_retry_connector import (
 
 from .adv_parser import SwitchbotSupportedType, parse_advertisement_data
 from .const import (
+    AirPurifierMode,
     FanMode,
     LockStatus,
     SwitchbotAccountConnectionError,
@@ -17,6 +18,7 @@ from .const import (
     SwitchbotAuthenticationError,
     SwitchbotModel,
 )
+from .devices.air_purifier import SwitchbotAirPurifier
 from .devices.base_light import SwitchbotBaseLight
 from .devices.blind_tilt import SwitchbotBlindTilt
 from .devices.bot import Switchbot
@@ -32,10 +34,12 @@ from .devices.lock import SwitchbotLock
 from .devices.plug import SwitchbotPlugMini
 from .devices.relay_switch import SwitchbotRelaySwitch
 from .devices.roller_shade import SwitchbotRollerShade
+from .devices.vacuum import SwitchbotVacuum
 from .discovery import GetSwitchbotDevices
 from .models import SwitchBotAdvertisement
 
 __all__ = [
+    "AirPurifierMode",
     "ColorMode",
     "FanMode",
     "GetSwitchbotDevices",
@@ -44,6 +48,7 @@ __all__ = [
     "Switchbot",
     "Switchbot",
     "SwitchbotAccountConnectionError",
+    "SwitchbotAirPurifier",
     "SwitchbotApiError",
     "SwitchbotAuthenticationError",
     "SwitchbotBaseLight",
@@ -66,6 +71,7 @@ __all__ = [
     "SwitchbotRollerShade",
     "SwitchbotSupportedType",
     "SwitchbotSupportedType",
+    "SwitchbotVacuum",
     "close_stale_connections",
     "close_stale_connections_by_address",
     "get_device",

{pyswitchbot-0.60.1 → pyswitchbot-0.62.0}/switchbot/adv_parser.py

@@ -10,6 +10,7 @@ from typing import Any, TypedDict
 from bleak.backends.device import BLEDevice
 from bleak.backends.scanner import AdvertisementData
 
+from .adv_parsers.air_purifier import process_air_purifier
 from .adv_parsers.blind_tilt import process_woblindtilt
 from .adv_parsers.bot import process_wohand
 from .adv_parsers.bulb import process_color_bulb
@@ -33,6 +34,7 @@ from .adv_parsers.relay_switch import (
 )
 from .adv_parsers.remote import process_woremote
 from .adv_parsers.roller_shade import process_worollershade
+from .adv_parsers.vacuum import process_vacuum, process_vacuum_k
 from .const import SwitchbotModel
 from .models import SwitchBotAdvertisement
 
@@ -237,6 +239,60 @@ SUPPORTED_TYPES: dict[str, SwitchbotSupportedType] = {
         "func": process_fan,
         "manufacturer_id": 2409,
     },
+    ".": {
+        "modelName": SwitchbotModel.K20_VACUUM,
+        "modelFriendlyName": "K20 Vacuum",
+        "func": process_vacuum,
+        "manufacturer_id": 2409,
+    },
+    "z": {
+        "modelName": SwitchbotModel.S10_VACUUM,
+        "modelFriendlyName": "S10 Vacuum",
+        "func": process_vacuum,
+        "manufacturer_id": 2409,
+    },
+    "3": {
+        "modelName": SwitchbotModel.K10_PRO_COMBO_VACUUM,
+        "modelFriendlyName": "K10+ Pro Combo Vacuum",
+        "func": process_vacuum,
+        "manufacturer_id": 2409,
+    },
+    "}": {
+        "modelName": SwitchbotModel.K10_VACUUM,
+        "modelFriendlyName": "K10+ Vacuum",
+        "func": process_vacuum_k,
+        "manufacturer_id": 2409,
+    },
+    "(": {
+        "modelName": SwitchbotModel.K10_PRO_VACUUM,
+        "modelFriendlyName": "K10+ Pro Vacuum",
+        "func": process_vacuum_k,
+        "manufacturer_id": 2409,
+    },
+    "*": {
+        "modelName": SwitchbotModel.AIR_PURIFIER,
+        "modelFriendlyName": "Air Purifier",
+        "func": process_air_purifier,
+        "manufacturer_id": 2409,
+    },
+    "+": {
+        "modelName": SwitchbotModel.AIR_PURIFIER,
+        "modelFriendlyName": "Air Purifier",
+        "func": process_air_purifier,
+        "manufacturer_id": 2409,
+    },
+    "7": {
+        "modelName": SwitchbotModel.AIR_PURIFIER,
+        "modelFriendlyName": "Air Purifier",
+        "func": process_air_purifier,
+        "manufacturer_id": 2409,
+    },
+    "8": {
+        "modelName": SwitchbotModel.AIR_PURIFIER,
+        "modelFriendlyName": "Air Purifier",
+        "func": process_air_purifier,
+        "manufacturer_id": 2409,
+    },
 }
 
 _SWITCHBOT_MODEL_TO_CHAR = {
@@ -285,10 +341,8 @@ def parse_advertisement_data(
             _mfr_id,
             model,
         )
-    except Exception as err:  # pylint: disable=broad-except
-        _LOGGER.exception(
-            "Failed to parse advertisement data: %s: %s", advertisement_data, err
-        )
+    except Exception:  # pylint: disable=broad-except
+        _LOGGER.exception("Failed to parse advertisement data: %s", advertisement_data)
         return None
 
     if not data:

pyswitchbot-0.62.0/switchbot/adv_parsers/air_purifier.py

@@ -0,0 +1,52 @@
+"""Air Purifier adv parser."""
+
+from __future__ import annotations
+
+import struct
+
+from ..const.air_purifier import AirPurifierMode, AirQualityLevel
+
+
+def process_air_purifier(
+    data: bytes | None, mfr_data: bytes | None
+) -> dict[str, bool | int]:
+    """Process air purifier services data."""
+    if mfr_data is None:
+        return {}
+    device_data = mfr_data[6:]
+
+    _seq_num = device_data[0]
+    _isOn = bool(device_data[1] & 0b10000000)
+    _mode = device_data[1] & 0b00000111
+    _is_aqi_valid = bool(device_data[2] & 0b00000100)
+    _child_lock = bool(device_data[2] & 0b00000010)
+    _speed = device_data[3] & 0b01111111
+    _aqi_level = (device_data[4] & 0b00000110) >> 1
+    _aqi_level = AirQualityLevel(_aqi_level).name.lower()
+    _work_time = struct.unpack(">H", device_data[5:7])[0]
+    _err_code = device_data[7]
+
+    return {
+        "isOn": _isOn,
+        "mode": get_air_purifier_mode(_mode, _speed),
+        "isAqiValid": _is_aqi_valid,
+        "child_lock": _child_lock,
+        "speed": _speed,
+        "aqi_level": _aqi_level,
+        "filter element working time": _work_time,
+        "err_code": _err_code,
+        "sequence_number": _seq_num,
+    }
+
+
+def get_air_purifier_mode(mode: int, speed: int) -> str | None:
+    if mode == 1:
+        if 0 <= speed <= 33:
+            return "level_1"
+        if 34 <= speed <= 66:
+            return "level_2"
+        return "level_3"
+    if 1 < mode <= 4:
+        mode += 2
+        return AirPurifierMode(mode).name.lower()
+    return None

{pyswitchbot-0.60.1 → pyswitchbot-0.62.0}/switchbot/adv_parsers/hub2.py

@@ -30,7 +30,7 @@ def process_wohub2(data: bytes | None, mfr_data: bytes | None) -> dict[str, Any]
     if _temp_c == 0 and humidity == 0:
         return {}
 
-    _wohub2_data = {
+    return {
         # Data should be flat, but we keep the original structure for now
         "temp": {"c": _temp_c, "f": _temp_f},
         "temperature": _temp_c,
@@ -40,8 +40,6 @@ def process_wohub2(data: bytes | None, mfr_data: bytes | None) -> dict[str, Any]
         "illuminance": calculate_light_intensity(light_level),
     }
 
-    return _wohub2_data
-
 
 def calculate_light_intensity(light_level: int) -> int:
     """

{pyswitchbot-0.60.1 → pyswitchbot-0.62.0}/switchbot/adv_parsers/hubmini_matter.py

@@ -28,10 +28,9 @@ def process_hubmini_matter(
     if _temp_c == 0 and humidity == 0:
         return {}
 
-    paraser_data = {
+    return {
         "temp": {"c": _temp_c, "f": _temp_f},
         "temperature": _temp_c,
         "fahrenheit": bool(temp_data[2] & 0b10000000),
         "humidity": humidity,
     }
-    return paraser_data

{pyswitchbot-0.60.1 → pyswitchbot-0.62.0}/switchbot/adv_parsers/meter.py

@@ -35,7 +35,7 @@ def process_wosensorth(data: bytes | None, mfr_data: bytes | None) -> dict[str,
     if _temp_c == 0 and humidity == 0 and battery == 0:
         return {}
 
-    _wosensorth_data = {
+    return {
         # Data should be flat, but we keep the original structure for now
         "temp": {"c": _temp_c, "f": _temp_f},
         "temperature": _temp_c,
@@ -44,8 +44,6 @@ def process_wosensorth(data: bytes | None, mfr_data: bytes | None) -> dict[str,
         "battery": battery,
     }
 
-    return _wosensorth_data
-
 
 def process_wosensorth_c(data: bytes | None, mfr_data: bytes | None) -> dict[str, Any]:
     """Process woSensorTH/Temp sensor services data with CO2."""

pyswitchbot-0.62.0/switchbot/adv_parsers/vacuum.py

@@ -0,0 +1,61 @@
+"""Vacuum parser."""
+
+from __future__ import annotations
+
+import struct
+
+
+def process_vacuum(
+    data: bytes | None, mfr_data: bytes | None
+) -> dict[str, bool | int | str]:
+    """Support for s10, k10+ pro combo, k20 process service data."""
+    if mfr_data is None:
+        return {}
+
+    _seq_num = mfr_data[6]
+    _soc_version = get_device_fw_version(mfr_data[8:11])
+    # Steps at the end of the last network configuration
+    _step = mfr_data[11] & 0b00001111
+    _mqtt_connected = bool(mfr_data[11] & 0b00010000)
+    _battery = mfr_data[12]
+    _work_status = mfr_data[13] & 0b00111111
+
+    return {
+        "sequence_number": _seq_num,
+        "soc_version": _soc_version,
+        "step": _step,
+        "mqtt_connected": _mqtt_connected,
+        "battery": _battery,
+        "work_status": _work_status,
+    }
+
+
+def get_device_fw_version(version_bytes: bytes) -> str | None:
+    version1 = version_bytes[0] & 0x0F
+    version2 = version_bytes[0] >> 4
+    version3 = struct.unpack("<H", version_bytes[1:])[0]
+    return f"{version1}.{version2}.{version3:>03d}"
+
+
+def process_vacuum_k(
+    data: bytes | None, mfr_data: bytes | None
+) -> dict[str, bool | int | str]:
+    """Support for k10+, k10+ pro process service data."""
+    if mfr_data is None:
+        return {}
+
+    _seq_num = mfr_data[6]
+    _dustbin_bound = bool(mfr_data[7] & 0b10000000)
+    _dusbin_connected = bool(mfr_data[7] & 0b01000000)
+    _network_connected = bool(mfr_data[7] & 0b00100000)
+    _work_status = (mfr_data[7] & 0b00010000) >> 4
+    _battery = mfr_data[8] & 0b01111111
+
+    return {
+        "sequence_number": _seq_num,
+        "dustbin_bound": _dustbin_bound,
+        "dusbin_connected": _dusbin_connected,
+        "network_connected": _network_connected,
+        "work_status": _work_status,
+        "battery": _battery,
+    }

{pyswitchbot-0.60.1 → pyswitchbot-0.62.0}/switchbot/const/__init__.py

@@ -3,10 +3,11 @@
 from __future__ import annotations
 
 from ..enum import StrEnum
-from .fan import FanMode as FanMode
+from .air_purifier import AirPurifierMode
+from .fan import FanMode
 
 # Preserve old LockStatus export for backwards compatibility
-from .lock import LockStatus as LockStatus
+from .lock import LockStatus
 
 DEFAULT_RETRY_COUNT = 3
 DEFAULT_RETRY_TIMEOUT = 1
@@ -67,3 +68,23 @@ class SwitchbotModel(StrEnum):
     ROLLER_SHADE = "Roller Shade"
     HUBMINI_MATTER = "HubMini Matter"
     CIRCULATOR_FAN = "Circulator Fan"
+    K20_VACUUM = "K20 Vacuum"
+    S10_VACUUM = "S10 Vacuum"
+    K10_VACUUM = "K10+ Vacuum"
+    K10_PRO_VACUUM = "K10+ Pro Vacuum"
+    K10_PRO_COMBO_VACUUM = "K10+ Pro Combo Vacuum"
+    AIR_PURIFIER = "Air Purifier"
+
+
+__all__ = [
+    "DEFAULT_RETRY_COUNT",
+    "DEFAULT_RETRY_TIMEOUT",
+    "DEFAULT_SCAN_TIMEOUT",
+    "AirPurifierMode",
+    "FanMode",
+    "LockStatus",
+    "SwitchbotAccountConnectionError",
+    "SwitchbotApiError",
+    "SwitchbotAuthenticationError",
+    "SwitchbotModel",
+]

pyswitchbot-0.62.0/switchbot/const/air_purifier.py

@@ -0,0 +1,23 @@
+from __future__ import annotations
+
+from enum import Enum
+
+
+class AirPurifierMode(Enum):
+    LEVEL_1 = 1
+    LEVEL_2 = 2
+    LEVEL_3 = 3
+    AUTO = 4
+    PET = 5
+    SLEEP = 6
+
+    @classmethod
+    def get_modes(cls) -> list[str]:
+        return [mode.name.lower() for mode in cls]
+
+
+class AirQualityLevel(Enum):
+    EXCELLENT = 0
+    GOOD = 1
+    MODERATE = 2
+    UNHEALTHY = 3

pyswitchbot-0.62.0/switchbot/devices/air_purifier.py

@@ -0,0 +1,142 @@
+"""Library to handle connection with Switchbot."""
+
+from __future__ import annotations
+
+import logging
+import struct
+from typing import Any
+
+from bleak.backends.device import BLEDevice
+
+from ..adv_parsers.air_purifier import get_air_purifier_mode
+from ..const import SwitchbotModel
+from ..const.air_purifier import AirPurifierMode, AirQualityLevel
+from .device import (
+    SwitchbotEncryptedDevice,
+    SwitchbotSequenceDevice,
+    update_after_operation,
+)
+
+_LOGGER = logging.getLogger(__name__)
+
+
+COMMAND_HEAD = "570f4c"
+COMMAND_TURN_OFF = f"{COMMAND_HEAD}010000"
+COMMAND_TURN_ON = f"{COMMAND_HEAD}010100"
+COMMAND_SET_MODE = {
+    AirPurifierMode.LEVEL_1.name.lower(): f"{COMMAND_HEAD}01010100",
+    AirPurifierMode.LEVEL_2.name.lower(): f"{COMMAND_HEAD}01010132",
+    AirPurifierMode.LEVEL_3.name.lower(): f"{COMMAND_HEAD}01010164",
+    AirPurifierMode.AUTO.name.lower(): f"{COMMAND_HEAD}01010200",
+    AirPurifierMode.PET.name.lower(): f"{COMMAND_HEAD}01010300",
+    AirPurifierMode.SLEEP.name.lower(): f"{COMMAND_HEAD}01010400",
+}
+DEVICE_GET_BASIC_SETTINGS_KEY = "570f4d81"
+
+
+class SwitchbotAirPurifier(SwitchbotSequenceDevice, SwitchbotEncryptedDevice):
+    """Representation of a Switchbot Air Purifier."""
+
+    def __init__(
+        self,
+        device: BLEDevice,
+        key_id: str,
+        encryption_key: str,
+        interface: int = 0,
+        model: SwitchbotModel = SwitchbotModel.AIR_PURIFIER,
+        **kwargs: Any,
+    ) -> None:
+        super().__init__(device, key_id, encryption_key, model, interface, **kwargs)
+
+    @classmethod
+    async def verify_encryption_key(
+        cls,
+        device: BLEDevice,
+        key_id: str,
+        encryption_key: str,
+        model: SwitchbotModel = SwitchbotModel.AIR_PURIFIER,
+        **kwargs: Any,
+    ) -> bool:
+        return await super().verify_encryption_key(
+            device, key_id, encryption_key, model, **kwargs
+        )
+
+    async def get_basic_info(self) -> dict[str, Any] | None:
+        """Get device basic settings."""
+        if not (_data := await self._get_basic_info()):
+            return None
+
+        _LOGGER.debug("data: %s", _data)
+        isOn = bool(_data[2] & 0b10000000)
+        version_info = (_data[2] & 0b00110000) >> 4
+        _mode = _data[2] & 0b00000111
+        isAqiValid = bool(_data[3] & 0b00000100)
+        child_lock = bool(_data[3] & 0b00000010)
+        _aqi_level = (_data[4] & 0b00000110) >> 1
+        aqi_level = AirQualityLevel(_aqi_level).name.lower()
+        speed = _data[6] & 0b01111111
+        pm25 = struct.unpack("<H", _data[12:14])[0] & 0xFFF
+        firmware = _data[15] / 10.0
+        mode = get_air_purifier_mode(_mode, speed)
+
+        return {
+            "isOn": isOn,
+            "version_info": version_info,
+            "mode": mode,
+            "isAqiValid": isAqiValid,
+            "child_lock": child_lock,
+            "aqi_level": aqi_level,
+            "speed": speed,
+            "pm25": pm25,
+            "firmware": firmware,
+        }
+
+    async def _get_basic_info(self) -> bytes | None:
+        """Return basic info of device."""
+        _data = await self._send_command(
+            key=DEVICE_GET_BASIC_SETTINGS_KEY, retry=self._retry_count
+        )
+
+        if _data in (b"\x07", b"\x00"):
+            _LOGGER.error("Unsuccessful, please try again")
+            return None
+
+        return _data
+
+    @update_after_operation
+    async def set_preset_mode(self, preset_mode: str) -> bool:
+        """Send command to set air purifier preset_mode."""
+        result = await self._send_command(COMMAND_SET_MODE[preset_mode])
+        return self._check_command_result(result, 0, {1})
+
+    @update_after_operation
+    async def turn_on(self) -> bool:
+        """Turn on the air purifier."""
+        result = await self._send_command(COMMAND_TURN_ON)
+        return self._check_command_result(result, 0, {1})
+
+    @update_after_operation
+    async def turn_off(self) -> bool:
+        """Turn off the air purifier."""
+        result = await self._send_command(COMMAND_TURN_OFF)
+        return self._check_command_result(result, 0, {1})
+
+    def get_current_percentage(self) -> Any:
+        """Return cached percentage."""
+        return self._get_adv_value("speed")
+
+    def is_on(self) -> bool | None:
+        """Return air purifier state from cache."""
+        return self._get_adv_value("isOn")
+
+    def get_current_aqi_level(self) -> Any:
+        """Return cached aqi level."""
+        return self._get_adv_value("aqi_level")
+
+    def get_current_pm25(self) -> Any:
+        """Return cached pm25."""
+        return self._get_adv_value("pm25")
+
+    def get_current_mode(self) -> Any:
+        """Return cached mode."""
+        return self._get_adv_value("mode")

{pyswitchbot-0.60.1 → pyswitchbot-0.62.0}/switchbot/devices/blind_tilt.py

@@ -100,8 +100,7 @@ class SwitchbotBlindTilt(SwitchbotBaseCover, SwitchbotSequenceDevice):
         """Send close command."""
         if self.get_position() > 50:
             return await self.close_up()
-        else:
-            return await self.close_down()
+        return await self.close_down()
 
     def get_position(self) -> Any:
         """Return cached tilt (0-100) of Blind Tilt."""

{pyswitchbot-0.60.1 → pyswitchbot-0.62.0}/switchbot/devices/device.py

@@ -646,7 +646,7 @@ class SwitchbotBaseDevice:
         """
         if not self._sb_adv_data:
             _LOGGER.exception("No advertisement data to update")
-            return
+            return None
         old_data = self._sb_adv_data.data.get("data") or {}
         merged_data = _merge_data(old_data, new_data)
         if merged_data == old_data:
@@ -688,9 +688,7 @@ class SwitchbotBaseDevice:
         ):
             return False
         time_since_last_full_update = time.monotonic() - self._last_full_update
-        if time_since_last_full_update < PASSIVE_POLL_INTERVAL:
-            return False
-        return True
+        return not time_since_last_full_update < PASSIVE_POLL_INTERVAL
 
 
 class SwitchbotDevice(SwitchbotBaseDevice):
@@ -723,11 +721,11 @@ class SwitchbotEncryptedDevice(SwitchbotDevice):
         """Switchbot base class constructor for encrypted devices."""
         if len(key_id) == 0:
             raise ValueError("key_id is missing")
-        elif len(key_id) != 2:
+        if len(key_id) != 2:
             raise ValueError("key_id is invalid")
         if len(encryption_key) == 0:
             raise ValueError("encryption_key is missing")
-        elif len(encryption_key) != 32:
+        if len(encryption_key) != 32:
             raise ValueError("encryption_key is invalid")
         self._key_id = key_id
         self._encryption_key = bytearray.fromhex(encryption_key)

{pyswitchbot-0.60.1 → pyswitchbot-0.62.0}/switchbot/devices/evaporative_humidifier.py

@@ -117,7 +117,7 @@ class SwitchbotEvaporativeHumidifier(SwitchbotEncryptedDevice):
         """Set device mode."""
         if mode == HumidifierMode.DRYING_FILTER:
             return await self.start_drying_filter()
-        elif mode not in MODES_COMMANDS:
+        if mode not in MODES_COMMANDS:
             raise ValueError("Invalid mode")
 
         command = COMMAND_SET_MODE + MODES_COMMANDS[mode]