chore(python_sdk): Fix linting issues after bumping PyLint

Author: tcamise-gpsw

demos/python/sdk_wireless_camera_control/open_gopro/features/streaming/stream_feature.py

@@ -113,7 +113,8 @@ async def start_stream(
 
         self._current = controller
         logger.info(f"Starting {stream_type.name.title()} stream")
-        match (result := await self._current.start(options)):
+        # Type checker can't narrow the union type based on runtime isinstance checks
+        match (result := await self._current.start(options)):  # type: ignore[arg-type]
             case Success():
                 logger.info("Stream started successfully.")
             case Failure():

demos/python/sdk_wireless_camera_control/open_gopro/gopro_base.py

@@ -301,9 +301,6 @@ class _InternalState(enum.IntFlag):
     def ip_address(self) -> str:
         """The IP address of the GoPro device
 
-        Raises:
-            GoProNotOpened: The GoPro IP address is not yet available
-
         Returns:
             str: IP address
         """
@@ -369,7 +366,7 @@ def _build_http_request_args(self, message: HttpMessage) -> dict[str, Any]:
         return request_args
 
     @enforce_message_rules
-    async def _get_json(
+    async def _get_json(  # type: ignore[override]
         self,
         message: HttpMessage,
         *,
@@ -407,7 +404,7 @@ async def _get_json(
         return response
 
     @enforce_message_rules
-    async def _get_stream(
+    async def _get_stream(  # type: ignore[override]
         self,
         message: HttpMessage,
         *,
@@ -433,7 +430,7 @@ async def _get_stream(
         return GoProResp(protocol=GoProResp.Protocol.HTTP, status=ErrorCode.SUCCESS, data=file, identifier=url)
 
     @enforce_message_rules
-    async def _put_json(
+    async def _put_json(  # type: ignore[override]
         self,
         message: HttpMessage,
         *,

demos/python/sdk_wireless_camera_control/open_gopro/gopro_wireless.py

@@ -848,17 +848,17 @@ def _handle_cohn(self, message: HttpMessage) -> HttpMessage:
         except GoProNotOpened:
             return message
 
-    async def _get_json(self, message: HttpMessage, *args: Any, **kwargs: Any) -> GoProResp:
+    async def _get_json(self, message: HttpMessage, *args: Any, **kwargs: Any) -> GoProResp:  # type: ignore[override]
         message = self._handle_cohn(message)
-        return await super()._get_json(*args, message=message, **kwargs)
+        return await super()._get_json(*args, message=message, **kwargs)  # type: ignore[call-arg]
 
-    async def _get_stream(self, message: HttpMessage, *args: Any, **kwargs: Any) -> GoProResp:
+    async def _get_stream(self, message: HttpMessage, *args: Any, **kwargs: Any) -> GoProResp:  # type: ignore[override]
         message = self._handle_cohn(message)
-        return await super()._get_stream(*args, message=message, **kwargs)
+        return await super()._get_stream(*args, message=message, **kwargs)  # type: ignore[call-arg]
 
-    async def _put_json(self, message: HttpMessage, *args: Any, **kwargs: Any) -> GoProResp:
+    async def _put_json(self, message: HttpMessage, *args: Any, **kwargs: Any) -> GoProResp:  # type: ignore[override]
         message = self._handle_cohn(message)
-        return await super()._put_json(*args, message=message, **kwargs)
+        return await super()._put_json(*args, message=message, **kwargs)  # type: ignore[call-arg]
 
     @GoProBase._ensure_opened((GoProMessageInterface.BLE,))
     async def _open_wifi(self, timeout: int = 30, retries: int = 5) -> None:

demos/python/sdk_wireless_camera_control/open_gopro/models/network_scan_responses.py

@@ -35,10 +35,10 @@
 class Hexlify(Adapter):
     """Construct adapter for pretty hex representation"""
 
-    def _decode(self, obj: bytes, context: Any, path: Any) -> str:
+    def _decode(self, obj: bytes, context: Any, path: Any) -> str:  # pylint: disable=unused-argument
         return obj.hex(":")
 
-    def _encode(self, obj: str, context: Any, path: Any) -> list[int]:
+    def _encode(self, obj: str, context: Any, path: Any) -> list[int]:  # pylint: disable=unused-argument
         return list(map(int, obj.split(":")))
 
 

demos/python/sdk_wireless_camera_control/open_gopro/network/ble/adapters/bleak_wrapper.py

@@ -14,6 +14,7 @@
 
 import bleak
 import pexpect
+from bleak.assigned_numbers import CharacteristicPropertyName
 from bleak.backends.characteristic import BleakGATTCharacteristic
 from bleak.backends.device import BLEDevice as BleakDevice
 from bleak.backends.scanner import AdvertisementData
@@ -344,11 +345,11 @@ async def discover_chars(self, handle: bleak.BleakClient, uuids: type[UUIDs] | N
             GattDB: Gatt Database
         """
 
-        def bleak_props_adapter(bleak_props: list[str]) -> CharProps:
+        def bleak_props_adapter(bleak_props: list[CharacteristicPropertyName]) -> CharProps:
             """Convert a list of bleak string properties into a CharProps
 
             Args:
-                bleak_props (list[str]): bleak strings to convert
+                bleak_props (list[CharacteristicPropertyName]): bleak strings to convert
 
             Returns:
                 CharProps: converted Enum

demos/python/sdk_wireless_camera_control/open_gopro/network/ble/services.py

@@ -180,7 +180,7 @@ class Characteristic:
     def __post_init__(self, init_descriptors: Optional[list[Descriptor]]) -> None:
         self._descriptors: dict[BleUUID, Descriptor] = {}
         # Mypy should eventually support this: see https://github.com/python/mypy/issues/3004
-        self.descriptors = init_descriptors or []  # type: ignore
+        self.descriptors = init_descriptors or []
         if self.descriptor_handle is None:
             self.descriptor_handle = self.handle + 1
 
@@ -296,7 +296,7 @@ class Service:
     def __post_init__(self, init_characteristics: Optional[list[Characteristic]]) -> None:
         self._characteristics: dict[BleUUID, Characteristic] = {}
         # Mypy should eventually support this: see https://github.com/python/mypy/issues/3004
-        self.characteristics = init_characteristics or []  # type: ignore
+        self.characteristics = init_characteristics or []
 
     def __str__(self) -> str:
         return self.name
@@ -408,7 +408,7 @@ def iter_items():
     def __init__(self, init_services: list[Service]) -> None:
         self._services: dict[BleUUID, Service] = {}
         # Mypy should eventually support this: see https://github.com/python/mypy/issues/3004
-        self.services = init_services  # type: ignore
+        self.services = init_services
         self.characteristics = self.CharacteristicView(self)
 
     @property

demos/python/sdk_wireless_camera_control/open_gopro/network/wifi/adapters/wireless.py

@@ -89,7 +89,7 @@ def __init__(self, interface: str | None = None, password: str | None = None) ->
         self._driver = self._detect_driver()
 
         # Attempt to set interface (will raise an exception if not able to auto-detect)
-        self.interface = interface  # type: ignore
+        self.interface = interface
 
         logger.debug(f"Wifi setup. Using {self}")
 
@@ -230,7 +230,7 @@ def interface(self, interface: str | None) -> None:
         Args:
             interface (str | None): interface (or None)
         """
-        self._driver.interface = interface  # type: ignore
+        self._driver.interface = interface
 
     @property
     def is_on(self) -> bool: