fix: remove unused imports and clean up code

Author: lachlangrose

loopstructural/gui/modelling/fault_adjacency_tab.py

@@ -5,7 +5,6 @@
     QPushButton,
     QTableWidget,
     QTableWidgetItem,
-    QTabWidget,
     QVBoxLayout,
     QWidget,
 )

loopstructural/gui/modelling/feature_details_panel.py

@@ -1,12 +1,10 @@
 from PyQt5.QtCore import Qt
 from PyQt5.QtWidgets import (
-    QCheckBox,
     QComboBox,
     QDoubleSpinBox,
     QFormLayout,
     QLabel,
     QScrollArea,
-    QSlider,
     QVBoxLayout,
     QWidget,
 )

loopstructural/gui/modelling/model_definition/dem.py

@@ -3,7 +3,6 @@
 from PyQt5.QtWidgets import QWidget
 from qgis.core import QgsMapLayerProxyModel
 from qgis.PyQt import uic
-from qgis.PyQt.QtWidgets import QSizePolicy
 
 
 class DEMWidget(QWidget):

loopstructural/gui/modelling/modelling_widget.py

@@ -1,5 +1,3 @@
-from xmlrpc.client import Fault
-
 from PyQt5.QtWidgets import QTabWidget, QVBoxLayout, QWidget
 
 from loopstructural.gui.modelling.fault_adjacency_tab import FaultAdjacencyTab
@@ -32,7 +30,9 @@ def __init__(
             self, data_manager=self.data_manager
         )
         self.fault_adjacency_tab_widget = FaultAdjacencyTab(self, data_manager=self.data_manager)
-        self.geological_model_tab_widget = GeologicalModelTab(self, model_manager=self.model_manager)
+        self.geological_model_tab_widget = GeologicalModelTab(
+            self, model_manager=self.model_manager
+        )
 
         mainLayout = QVBoxLayout(self)
         self.setLayout(mainLayout)

loopstructural/gui/visualisation/feature_list_widget.py

@@ -1,6 +1,5 @@
 from typing import Optional, Union
 
-from PyQt5.QtCore import Qt
 from PyQt5.QtWidgets import QMenu, QPushButton, QTreeWidget, QTreeWidgetItem, QVBoxLayout, QWidget
 
 from LoopStructural.datatypes import VectorPoints

loopstructural/gui/visualisation/object_list_widget.py

@@ -185,7 +185,7 @@ def remove_selected_object(self):
             if self.viewer and hasattr(self.viewer, 'remove_object'):
                 self.viewer.remove_object(object_label)
             else:
-                print(f"Error: Viewer is not initialized or does not support object removal.")
+                print("Error: Viewer is not initialized or does not support object removal.")
             self.treeWidget.takeTopLevelItem(self.treeWidget.indexOfTopLevelItem(item))
 
     def show_add_object_menu(self):

loopstructural/main/model_manager.py

@@ -1,6 +1,4 @@
 from collections import defaultdict
-from collections.abc import Callable
-from operator import ne
 from typing import Callable
 
 import geopandas as gpd
@@ -228,8 +226,7 @@ def update_foliation_features(self):
         This method will automatically add unconformities based on the stratigraphic column.
         """
         stratigraphic_column = {}
-        unit_id = 0
-        for i, group in enumerate(reversed(self.stratigraphic_column.get_groups())):
+        for _i, group in enumerate(reversed(self.stratigraphic_column.get_groups())):
             val = 0
             data = []
             groupname = group.name
@@ -321,7 +318,7 @@ def valid(self):
         if len(self.stratigraphy) == 0:
             valid = False
         if len(self.faults) > 0:
-            for fault_name, fault_data in self.faults.items():
+            for _fault_name, fault_data in self.faults.items():
                 if 'data' in fault_data and not fault_data['data'].empty:
                     valid = True
                 else:

loopstructural/toolbelt/preferences.py

@@ -5,8 +5,6 @@
 # standard
 from dataclasses import asdict, dataclass, fields
 
-from numpy import interp
-
 # PyQGIS
 from qgis.core import QgsSettings