Add ruff fixes

Author: remcom

custom_components/victron/binary_sensor.py

@@ -2,22 +2,18 @@
 
 from __future__ import annotations
 
-from contextlib import suppress
 from typing import cast
 
 from homeassistant.core import HomeAssistant, HassJob
 
-from collections.abc import Callable
-from homeassistant.helpers.typing import StateType
 
 from dataclasses import dataclass
 
 from homeassistant.helpers.update_coordinator import CoordinatorEntity
 
 from homeassistant.config_entries import ConfigEntry
-from homeassistant.core import HomeAssistant
 from homeassistant.helpers.entity_platform import AddEntitiesCallback
-from homeassistant.helpers import event, entity
+from homeassistant.helpers import entity
 
 from homeassistant.components.binary_sensor import (
     BinarySensorEntityDescription,
@@ -27,7 +23,7 @@
 
 from .coordinator import victronEnergyDeviceUpdateCoordinator
 from .base import VictronBaseEntityDescription
-from .const import DOMAIN, CONF_ADVANCED_OPTIONS, register_info_dict, BoolReadEntityType
+from .const import DOMAIN, register_info_dict, BoolReadEntityType
 
 import logging
 

custom_components/victron/button.py

@@ -9,7 +9,6 @@
 from homeassistant.helpers.update_coordinator import CoordinatorEntity
 
 from homeassistant.config_entries import ConfigEntry
-from homeassistant.core import HomeAssistant
 from homeassistant.helpers.entity_platform import AddEntitiesCallback
 from homeassistant.helpers import entity
 

custom_components/victron/config_flow.py

@@ -21,7 +21,7 @@
 
 from homeassistant.core import callback
 
-from homeassistant.config_entries import ConfigEntry, ConfigFlow, OptionsFlow
+from homeassistant.config_entries import ConfigEntry
 
 from .const import (
     DOMAIN,
@@ -129,7 +129,7 @@ async def async_step_user(
             # not yet working
             await self.async_set_unique_id("victron")
             self._abort_if_unique_id_configured()
-        except Exception as e:
+        except Exception:
             errors["base"] = "already_configured"
             already_configured = True
 

custom_components/victron/const.py

@@ -3,7 +3,6 @@
 from enum import Enum
 from homeassistant.const import (
     PERCENTAGE,
-    UnitOfPower,
     UnitOfEnergy,
     UnitOfElectricPotential,
     UnitOfElectricCurrent,

custom_components/victron/coordinator.py

@@ -11,7 +11,7 @@
 
 from pymodbus.constants import Endian
 from pymodbus.payload import BinaryPayloadDecoder
-from pymodbus.pdu import register_read_message
+from pymodbus.pdu.register_message import ReadHoldingRegistersResponse
 
 from datetime import timedelta
 
@@ -106,7 +106,7 @@ async def _async_update_data(self) -> dict:
     def parse_register_data(
         self,
         buffer: ReadHoldingRegistersResponse,
-        registerInfo: OrderedDict(str, RegisterInfo),
+        registerInfo: OrderedDict[str, RegisterInfo],
         unit: int,
     ) -> dict:
         decoder = BinaryPayloadDecoder.fromRegisters(
@@ -184,7 +184,7 @@ async def fetch_registers(self, unit, registerData):
     def write_register(self, unit, address, value):
         # try:
 
-        resp = self.api_write(unit, address, value)
+        self.api_write(unit, address, value)
 
     # except:
     # TODO raise specific write error

custom_components/victron/hub.py

@@ -1,5 +1,4 @@
 from pymodbus.client import ModbusTcpClient
-from pymodbus.exceptions import ModbusException
 
 import threading
 from collections import OrderedDict

custom_components/victron/number.py

@@ -2,9 +2,7 @@
 
 from __future__ import annotations
 
-from collections.abc import Callable
 from dataclasses import dataclass
-from typing import Optional, cast
 
 from homeassistant import config_entries
 from homeassistant.components.number import (
@@ -19,15 +17,12 @@
     UnitOfElectricPotential,
     UnitOfPower,
     UnitOfElectricCurrent,
-    UnitOfTime,
 )
 
-import math
 
-from homeassistant.core import HomeAssistant, callback
+from homeassistant.core import HomeAssistant
 from homeassistant.helpers.entity import EntityCategory
 from homeassistant.helpers.entity_platform import AddEntitiesCallback
-from homeassistant.helpers.update_coordinator import CoordinatorEntity
 
 from .coordinator import victronEnergyDeviceUpdateCoordinator
 from .base import VictronWriteBaseEntityDescription
@@ -45,7 +40,6 @@
     CONF_AC_CURRENT_LIMIT,
 )
 
-from homeassistant.helpers.typing import StateType
 from homeassistant.helpers import entity
 
 import logging

custom_components/victron/select.py

@@ -22,12 +22,10 @@
 from .const import DOMAIN, register_info_dict, SelectWriteType, CONF_ADVANCED_OPTIONS
 from .base import VictronWriteBaseEntityDescription
 
-from collections.abc import Callable
-from homeassistant.helpers.typing import StateType
 
 from datetime import timedelta
 from homeassistant.util import utcnow
-from homeassistant.helpers import event, entity
+from homeassistant.helpers import event
 
 import logging
 

custom_components/victron/sensor.py

@@ -4,9 +4,7 @@
 
 import logging
 
-from datetime import timedelta
-from homeassistant.util import utcnow
-from homeassistant.helpers import event, entity
+from homeassistant.helpers import entity
 from homeassistant.core import HomeAssistant, HassJob, callback
 from homeassistant.helpers.entity_platform import AddEntitiesCallback
 from homeassistant.helpers.update_coordinator import CoordinatorEntity
@@ -43,8 +41,6 @@
     UnitOfPressure,
 )
 
-from collections.abc import Callable
-from homeassistant.helpers.typing import StateType
 
 _LOGGER = logging.getLogger(__name__)
 

custom_components/victron/switch.py

@@ -22,8 +22,6 @@
 from .const import DOMAIN, register_info_dict, SwitchWriteType, CONF_ADVANCED_OPTIONS
 from .base import VictronWriteBaseEntityDescription
 
-from collections.abc import Callable
-from homeassistant.helpers.typing import StateType
 
 import logging