tests/conftest.py (1)

Line range hint 4-4: The import pytestqt is unused in this file.

Consider removing the import if it's not needed to avoid unnecessary dependencies.

sleap/util.py (2)

Line range hint 96-96: Avoid using bare except statements as they can catch unexpected exceptions and make debugging difficult. Specify the exception type.

-    except:
+    except Exception as e:
+        # Handle specific exception e

---

Line range hint 130-130: The variable name field in the loop shadows the import from attrs. Consider renaming the variable to avoid confusion and potential bugs.

-    for field in attr.fields(cls):
+    for attr_field in attr.fields(cls):

tests/gui/test_commands.py (4)

Line range hint 228-228: Avoid equality comparisons to True; use if okay: for truth checks.

- if okay == True:
+ if okay:

---

Line range hint 361-361: The local variable last_lf_frame is assigned but never used, which could indicate redundant or incomplete logic.

---

Line range hint 369-369: For boolean checks, directly use the condition without comparing to True.

- if video.backend.grayscale == True:
+ if video.backend.grayscale:

---

Line range hint 529-529: This f-string does not contain any placeholders, which suggests it might be incorrectly formatted or unnecessary.

tests/io/test_dataset.py (3)

Line range hint 692-692: The variable new_labels_json is defined but not used.

Consider removing the unused variable or using it if it was intended for further operations.

---

Line range hint 708-708: The variable new_labels_json is defined but not used.

Consider removing the unused variable or using it if it was intended for further operations.

---

Line range hint 1495-1495: The variable track_insts is defined but not used.

Consider removing the unused variable or using it if it was intended for further operations.

sleap/io/video.py (6)

Line range hint 231-231: Avoid using a bare except statement; specify the exception type to improve error handling.

-        except:
+        except Exception as e:

---

Line range hint 242-242: Avoid using a bare except statement; specify the exception type to improve error handling.

-        except:
+        except Exception as e:

---

Line range hint 107-107: Use isinstance() for type checking instead of comparing types directly.

-        if type(self.filename) is str:
+        if isinstance(self.filename, str):

---

Line range hint 536-536: Use isinstance() for type checking instead of comparing types directly.

-        if type(self.filename) is str:
+        if isinstance(self.filename, str):

---

Line range hint 959-959: Remove unnecessary f-string as it does not contain any placeholders.

-        f"Cannot find a video file: {path}"
+        "Cannot find a video file: " + path

---

Line range hint 1487-1487: Undefined name sleap in the method to_pipeline. Ensure that the necessary module is imported or available in the scope.

+ import sleap

sleap/instance.py (3)

Line range hint 89-89: Undefined name self used in method definition.

- def numpy() -> np.ndarray:
+ def numpy(self) -> np.ndarray:

The method numpy is missing the self parameter which is necessary for instance methods in Python classes. This should be added to avoid runtime errors.

---

Line range hint 411-411: Use isinstance() for type checks.

- if from_predicted is not None and type(from_predicted) != PredictedInstance:
+ if from_predicted is not None and not isinstance(from_predicted, PredictedInstance):

Replace type comparison with isinstance() for better practice and to support polymorphism.

---

Line range hint 1631-1631: Undefined name Labels.

The name Labels is used but not defined in this scope. Ensure that it is imported or defined before use, or correct the name if it's a typo.

sleap/gui/app.py (1)

Line range hint 264-264: The local variable filename is assigned but never used in the method dragEnterEvent.

- filename = event.mimeData().data(mime_format).data().decode()

sleap/gui/widgets/video.py (3)

Line range hint 27-75: Move all module-level imports to the top of the file.

+ from collections import deque
+ import atexit
+ import math
+ import time
+ from typing import Callable, List, Optional, Union
+ import numpy as np
+ import qimage2ndarray
+ from qtpy import QtCore, QtWidgets
+ from qtpy.QtCore import QLineF, QMarginsF, QPointF, QRectF, Qt
+ from qtpy.QtGui import (
+     QBrush,
+     QColor,
+     QCursor,
+     QFont,
+     QImage,
+     QKeyEvent,
+     QKeySequence,
+     QMouseEvent,
+     QPainter,
+     QPainterPath,
+     QPen,
+     QPixmap,
+     QPolygonF,
+     QTransform,
+ )
+ from qtpy.QtWidgets import (
+     QApplication,
+     QGraphicsEllipseItem,
+     QGraphicsItem,
+     QGraphicsObject,
+     QGraphicsPolygonItem,
+     QGraphicsRectItem,
+     QGraphicsScene,
+     QGraphicsTextItem,
+     QGraphicsView,
+     QShortcut,
+     QVBoxLayout,
+     QWidget,
+ )
+ import sleap
+ from sleap.gui.color import ColorManager
+ from sleap.gui.shortcuts import Shortcuts
+ from sleap.gui.state import GuiState
+ from sleap.gui.widgets.slider import VideoSlider
+ from sleap.instance import Instance, Point, PredictedInstance
+ from sleap.io.video import Video
+ from sleap.prefs import prefs
+ from sleap.skeleton import Node

---

Line range hint 306-306: Replace the bare except with a more specific exception type to avoid catching unexpected exceptions.

-        except:
+        except Exception as e:
+            # Handle specific exceptions or log error
+            print(f"Error while loading frame: {e}")

---

Line range hint 1470-1470: The local variable in_bounds is assigned but never used. Consider removing it if it's unnecessary.

-        in_bounds = True

sleap/io/dataset.py (10)

Line range hint 61-61: Unused import h5py.

Consider removing the unused import to clean up the code and improve readability.

---

Line range hint 69-69: Avoid using bare except.

Using a bare except: can catch unexpected exceptions and hide bugs. Specify the exception type to improve error handling.

---

Line range hint 952-952: Use isinstance() for type checks.

Replace type comparison using type() with isinstance() for a more robust and idiomatic check.

- if type(inst) == PredictedInstance:
+ if isinstance(inst, PredictedInstance):

---

Line range hint 1465-1465: Use isinstance() for type checks.

Replace type comparison using type() with isinstance() for a more robust and idiomatic check.

- if type(inst) == Instance:
+ if isinstance(inst, Instance):

---

Line range hint 2365-2365: Undefined name glob.

The glob module is used but not imported. Ensure to import glob from the standard library to fix the undefined name error.

---

Line range hint 2460-2460: Avoid equality comparisons to False.

Use if not ret: instead of if ret == False: for a more Pythonic approach.

- if ret == False:
+ if not ret:

---

Line range hint 2548-2548: Avoid equality comparisons to False.

Use if not ret: instead of if ret == False: for a more Pythonic approach.

- if ret == False:
+ if not ret:

---

Line range hint 2563-2563: Undefined name sleap.

The sleap module is referenced but not defined or imported in this scope. Ensure that it is correctly imported or defined to avoid runtime errors.

---

Line range hint 2653-2653: Use isinstance() for type checks.

Replace type comparison using type() with isinstance() for a more robust and idiomatic check.

- if type(video) == int:
+ if isinstance(video, int):

---

Line range hint 2658-2658: F-string without placeholders.

The f-string is used without any placeholders. This can be changed to a regular string if no formatting is needed.

- raise IndexError(f"There are no videos in this project. No points matrix to return.")
+ raise IndexError("There are no videos in this project. No points matrix to return.")

sleap/gui/commands.py (4)

Line range hint 196-196: Undefined name MainWindow used.

- MainWindow
+ self.app.__class__()  # Assuming this is intended to reference the main window class.

---

Line range hint 860-860: Local variable file_dir is assigned to but never used.

Consider removing the unused variable to clean up the code.

---

Line range hint 2545-2545: f-string without any placeholders.

- f"This video type {type(video.backend)} for video at index {idx} does not support grayscale yet."
+ "This video type does not support grayscale yet."

---

Line range hint 2877-2877: f-string without any placeholders.

- f"Unable to load {filename}."
+ "Unable to load the file."