From 632e0926195ae639c41a193a641408496abbb806 Mon Sep 17 00:00:00 2001
From: Pierre-Antoine Rouby <pierre-antoine.rouby@inrae.fr>
Date: Tue, 25 Apr 2023 14:03:25 +0200
Subject: [PATCH] geometry: Delete dead code and minor change.

---
 src/Model/Geometry/ProfileXYZ.py              |  6 -----
 src/View/Geometry/GeometryWindow.py           |  4 +--
 src/View/Geometry/Profile/ProfileWindow.py    |  4 +--
 .../Geometry/Profile/qtableview_profile.py    | 27 +++++++++++--------
 src/View/Geometry/qtableview_reach.py         | 25 +++++++----------
 5 files changed, 29 insertions(+), 37 deletions(-)

diff --git a/src/Model/Geometry/ProfileXYZ.py b/src/Model/Geometry/ProfileXYZ.py
index 1118cc00..a495a306 100644
--- a/src/Model/Geometry/ProfileXYZ.py
+++ b/src/Model/Geometry/ProfileXYZ.py
@@ -1,7 +1,6 @@
 # -*- coding: utf-8 -*-
 
 import numpy as np
-import pandas as pd
 from typing import List
 
 from Model.Geometry.Profile import Profile
@@ -35,11 +34,6 @@ class ProfileXYZ(Profile):
             reach = reach,
         )
 
-    def __repr__(self):
-        df = pd.DataFrame(columns=["X", "Y", "Z", "Name"],
-                          data=[[p.x, p.y, p.z, p.name] for p in self._points])
-        return f"\nProfileXYZ : {self.name}\n{df}"
-
     def x(self):
         return [point.x for point in self._points]
 
diff --git a/src/View/Geometry/GeometryWindow.py b/src/View/Geometry/GeometryWindow.py
index 89f3be26..aee955a0 100644
--- a/src/View/Geometry/GeometryWindow.py
+++ b/src/View/Geometry/GeometryWindow.py
@@ -71,7 +71,7 @@ class GeometryWindow(QMainWindow, WindowToolKit):
         self.paste_sc = QShortcut(QKeySequence.Paste, self)
 
     def setup_model(self):
-        self._tablemodel = PandasModelEditable(
+        self._tablemodel = TableEditableModel(
             headers = self.ui.tableView_header,
             reach = self._reach,
             undo = self._undo_stack
@@ -435,6 +435,6 @@ class GeometryWindow(QMainWindow, WindowToolKit):
                     buf.append(row)
 
                 self._tablemodel = None
-                self._tablemodel = PandasModelEditable(buf)
+                self._tablemodel = TableEditableModel(buf)
                 self.tableView.setModel(self._tablemodel)
                 filename = ''
diff --git a/src/View/Geometry/Profile/ProfileWindow.py b/src/View/Geometry/Profile/ProfileWindow.py
index c7f574fa..241a3b2f 100644
--- a/src/View/Geometry/Profile/ProfileWindow.py
+++ b/src/View/Geometry/Profile/ProfileWindow.py
@@ -81,7 +81,7 @@ class ProfileWindow(QMainWindow):
         self.paste_sc = QShortcut(QKeySequence.Paste, self)
 
     def setup_model(self):
-        self._model = PandasModelEditable(
+        self._model = TableEditableModel(
             profile = self._profile,
             undo = self._undo_stack
         )
@@ -266,7 +266,7 @@ class ProfileWindow(QMainWindow):
                     buf.append(row)
 
                 self._model = None
-                self._model = PandasModelEditable(buf)
+                self._model = TableEditableModel(buf)
                 self.ui.tableView.setModel(self._model)
                 self.fileName = ''
 
diff --git a/src/View/Geometry/Profile/qtableview_profile.py b/src/View/Geometry/Profile/qtableview_profile.py
index f2a25f26..6c794434 100644
--- a/src/View/Geometry/Profile/qtableview_profile.py
+++ b/src/View/Geometry/Profile/qtableview_profile.py
@@ -1,11 +1,16 @@
 # -*- coding: utf-8 -*-
 
 import numpy as np
-from PyQt5.QtGui import QFont
-
-from PyQt5.QtWidgets import QMessageBox
-from PyQt5 import QtWidgets, QtGui
-from PyQt5.QtCore import QModelIndex, Qt, QAbstractTableModel, QVariant, QCoreApplication
+from PyQt5.QtGui import (
+    QFont, QColor
+)
+from PyQt5.QtWidgets import (
+    QMessageBox, QStyledItemDelegate, QLineEdit
+)
+from PyQt5.QtCore import (
+    QModelIndex, Qt, QAbstractTableModel,
+    QVariant, QCoreApplication
+)
 
 from Model.Geometry.ProfileXYZ import ProfileXYZ
 
@@ -14,7 +19,7 @@ from View.Geometry.Profile.ProfileUndoCommand import *
 _translate = QCoreApplication.translate
 
 
-class PandasModelEditable(QAbstractTableModel):
+class TableEditableModel(QAbstractTableModel):
     def __init__(self, profile: ProfileXYZ, header=None, undo=None):
         QAbstractTableModel.__init__(self)
 
@@ -68,9 +73,9 @@ class PandasModelEditable(QAbstractTableModel):
                 value = self._profile.point(index.row()).z
                 if role == Qt.ForegroundRole:
                     if value == self._profile.z_min():
-                        return QtGui.QColor("red")
+                        return QColor("red")
                     elif value == self._profile.z_max():
-                        return QtGui.QColor("blue")
+                        return QColor("blue")
 
                 if role == Qt.ToolTipRole:
                     if value == self._profile.z_min():
@@ -92,7 +97,7 @@ class PandasModelEditable(QAbstractTableModel):
                         return font
 
                     if role == Qt.ForegroundRole:
-                        return QtGui.QColor("darkRed")
+                        return QColor("darkRed")
 
                     if role == Qt.ToolTipRole:
                         if value.strip().upper() == "RG":
@@ -280,14 +285,14 @@ class PandasModelEditable(QAbstractTableModel):
         self.layoutChanged.emit()
 
 
-class Delegate(QtWidgets.QStyledItemDelegate):
+class Delegate(QStyledItemDelegate):
     def __init__(self, parent=None, setModelDataEvent=None):
         super(Delegate, self).__init__(parent)
         self.setModelDataEvent = setModelDataEvent
 
     def createEditor(self, parent, option, index):
         index.model().data(index, Qt.DisplayRole)
-        return QtWidgets.QLineEdit(parent)
+        return QLineEdit(parent)
 
     def setEditorData(self, editor, index):
         value = index.model().data(index, Qt.DisplayRole)
diff --git a/src/View/Geometry/qtableview_reach.py b/src/View/Geometry/qtableview_reach.py
index 7f0e4988..3622ceb8 100644
--- a/src/View/Geometry/qtableview_reach.py
+++ b/src/View/Geometry/qtableview_reach.py
@@ -1,18 +1,9 @@
 # -*- coding: utf-8 -*-
 
 import time
-from functools import reduce
-from itertools import groupby
-from operator import itemgetter
 
-import numpy as np
-import pandas as pd
-
-from PyQt5 import (
-    QtGui, QtWidgets,
-)
 from PyQt5.QtGui import (
-    QKeySequence,
+    QKeySequence, QColor
 )
 from PyQt5.QtCore import (
     Qt, QAbstractTableModel, QModelIndex,
@@ -20,6 +11,8 @@ from PyQt5.QtCore import (
 )
 from PyQt5.QtWidgets import (
     QMessageBox, QUndoCommand, QUndoStack,
+    QStyledItemDelegate, QLineEdit, QAbstractItemView,
+    QComboBox,
 )
 
 from Model.Geometry import Reach
@@ -29,7 +22,7 @@ from View.Geometry.ReachUndoCommand import *
 _translate = QCoreApplication.translate
 
 
-class PandasModelEditable(QAbstractTableModel):
+class TableEditableModel(QAbstractTableModel):
     def __init__(self, reach, headers=None, undo=None):
         QAbstractTableModel.__init__(self)
         data_list = []
@@ -239,14 +232,14 @@ class PandasModelEditable(QAbstractTableModel):
         self.layoutChanged.emit()
 
 
-class Delegate(QtWidgets.QStyledItemDelegate):
+class Delegate(QStyledItemDelegate):
     def __init__(self, parent=None, setModelDataEvent=None):
         super(Delegate, self).__init__(parent)
         self.setModelDataEvent = setModelDataEvent
 
     def createEditor(self, parent, option, index):
         index.model().data(index, Qt.DisplayRole)
-        return QtWidgets.QLineEdit(parent)
+        return QLineEdit(parent)
 
     def setEditorData(self, editor, index):
         value = index.model().data(index, Qt.DisplayRole)
@@ -261,18 +254,18 @@ class Delegate(QtWidgets.QStyledItemDelegate):
     def updateEditorGeometry(self, editor, option, index):
         editor.setGeometry(option.rect)
 
-class Delegate1(QtWidgets.QStyledItemDelegate):
+class Delegate1(QStyledItemDelegate):
     def __init__(self, owner, choices):
         super().__init__(owner)
         self.items = choices
 
     def paint(self, painter, option, index):
-        if isinstance(self.parent(), QtWidgets.QAbstractItemView):
+        if isinstance(self.parent(), QAbstractItemView):
             self.parent().openPersistentEditor(index)
         super(Delegate1, self).paint(painter, option, index)
 
     def createEditor(self, parent, option, index):
-        editor = QtWidgets.QComboBox(parent)
+        editor = QComboBox(parent)
         # editor.currentIndexChanged.connect(self.commit_editor)
         editor.addItems(self.items)
         return editor
-- 
GitLab