Fix some alerts from the LGTM system (#11143)

Signed-off-by: Taras Drozdovskyi <[email protected]>

Author: tdrozdovsky

scripts/tools/memory/memdf/df.py

@@ -32,7 +32,8 @@ def __init__(self, *args, **kwargs):
                 self[c] = pd.Series()
         types = {c: self.dtype[c] for c in self.columns if c in self.dtype}
         typed_columns = list(types.keys())
-        self[typed_columns] = self.astype(types, copy=False)[typed_columns]
+        self[typed_columns] = self.astype(types, copy=False)[
+            typed_columns]  # lgtm [py/hash-unhashable-value]
         self.attrs['name'] = self.name
 
 

scripts/tools/memory/report_summary.py

@@ -28,8 +28,6 @@
 
 import sys
 
-import numpy  # type: ignore
-
 import memdf.collect
 import memdf.report
 import memdf.select

src/controller/python/chip/ChipBleUtility.py

@@ -24,7 +24,7 @@
 
 from __future__ import absolute_import
 from __future__ import print_function
-from ctypes import *
+from ctypes import *  # lgtm [py/polluting-import]
 from .ChipUtility import ChipUtility
 
 

src/controller/python/chip/ChipBluezMgr.py

@@ -35,7 +35,7 @@
 import uuid
 import queue
 
-from ctypes import *
+from ctypes import CFUNCTYPE, PYFUNCTYPE, c_void_p, c_int
 
 try:
     from gi.repository import GObject

src/controller/python/chip/ChipCommissionableNodeCtrl.py

@@ -25,7 +25,7 @@
 
 from __future__ import absolute_import
 from __future__ import print_function
-from ctypes import *
+from ctypes import c_void_p, c_uint32
 from .ChipStack import *
 from .exceptions import *
 

src/controller/python/chip/ChipCoreBluetoothMgr.py

@@ -24,7 +24,7 @@
 
 from __future__ import absolute_import
 from __future__ import print_function
-from ctypes import *
+from ctypes import *  # lgtm [py/polluting-import]
 from Foundation import *
 
 import logging

src/controller/python/chip/ChipDeviceCtrl.py

@@ -28,7 +28,7 @@
 from __future__ import absolute_import
 from __future__ import print_function
 import asyncio
-from ctypes import *
+from ctypes import *  # lgtm [py/polluting-import]
 from .ChipStack import *
 from .interaction_model import delegate as im
 from .exceptions import *
@@ -334,11 +334,11 @@ def DeviceAvailableCallback(device, err):
 
         # The callback might have been received synchronously (during self._ChipStack.Call()).
         # Check if the device is already set before waiting for the callback.
-        if returnDevice.value == None:
+        if returnDevice.value is None:
             with deviceAvailableCV:
                 deviceAvailableCV.wait()
 
-        if returnDevice.value == None:
+        if returnDevice.value is None:
             raise self._ChipStack.ErrorToException(CHIP_ERROR_INTERNAL)
         return returnDevice
 

src/controller/python/chip/ChipStack.py

@@ -33,7 +33,7 @@
 import platform
 import logging
 from threading import Lock, Event, Condition
-from ctypes import *
+from ctypes import *  # lgtm [py/polluting-import]
 from .ChipUtility import ChipUtility
 from .exceptions import *
 

src/controller/python/chip/ChipUtility.py

@@ -24,7 +24,7 @@
 from __future__ import absolute_import
 from __future__ import print_function
 import binascii
-from ctypes import *
+from ctypes import c_void_p, c_byte
 
 
 class ChipUtility(object):

src/controller/python/chip/ble/library_handle.py

@@ -15,7 +15,7 @@
 #
 
 import chip.native
-import ctypes
+import ctypes  # lgtm [py/import-and-import-from]
 from ctypes import c_bool, c_void_p, c_char_p, c_uint32, py_object
 from chip.ble.types import DeviceScannedCallback, ScanDoneCallback
 

src/controller/python/chip/clusters/Command.py

@@ -16,7 +16,7 @@
 #
 
 from asyncio.futures import Future
-import ctypes
+import ctypes  # lgtm [py/import-and-import-from]
 from dataclasses import dataclass
 from typing import Type
 from ctypes import CFUNCTYPE, c_char_p, c_size_t, c_void_p, c_uint32, c_uint16, c_uint8, py_object
@@ -74,7 +74,7 @@ def _handleResponse(self, path: CommandPath, status: Status, response: bytes):
             self._future.set_result(None)
         else:
             # If a type hasn't been assigned, let's auto-deduce it.
-            if (self._expect_type == None):
+            if (self._expect_type is None):
                 self._expect_type = FindCommandClusterObject(False, path)
 
             if self._expect_type:

src/controller/python/chip/clusters/__init__.py

@@ -23,5 +23,5 @@
 """Provides Python APIs for CHIP."""
 from . import Command
 from . import Attribute
-from .Objects import *
+from .Objects import *  # lgtm [py/polluting-import]
 from . import CHIPClusters

src/controller/python/chip/interaction_model/__init__.py

@@ -26,7 +26,7 @@
 
 from chip.exceptions import ChipStackException
 
-__all__ = ["IMDelegate", "Status", "InteractionModelError"]
+__all__ = ["Status", "InteractionModelError"]
 
 
 class Status(enum.IntEnum):

src/controller/python/chip/interaction_model/delegate.py

@@ -17,7 +17,7 @@
 from abc import abstractmethod
 from construct import Struct, Int64ul, Int32ul, Int16ul, Int8ul
 from ctypes import CFUNCTYPE, c_void_p, c_uint32, c_uint64, c_uint8, c_uint16, c_ssize_t
-import ctypes
+import ctypes  # lgtm [py/import-and-import-from]
 import chip.native
 import threading
 import chip.tlv