code tidy

Author: nickzoic

countess/core/parameters.py

@@ -166,10 +166,10 @@ class NumericParam(ScalarWithOperatorsParam):
     def set_value(self, value):
         try:
             if type(value) is str:
-                if m := re.match(r'(\d+(\.\d*)?)', value):
+                if m := re.match(r"(\d+(\.\d*)?)", value):
                     value = m.group(1)
             self._value = self.var_type(value)
-        except (TypeError, ValueError) as exc:
+        except (TypeError, ValueError):
             self.reset_value()
 
     # Operator methods which apply only to numerics

countess/gui/config.py

@@ -401,7 +401,7 @@ def value_changed_callback(self, *_):
             else:
                 self.var.set(self.set_value(self.var.get()))
         else:
-            val = self.set_value(self.var.get())
+            self.set_value(self.var.get())
 
     def widget_modified_callback(self, *_):
         # only gets called the *first* time a modification happens, unless

countess/plugins/correlation.py

@@ -4,7 +4,7 @@
 from duckdb import DuckDBPyConnection, DuckDBPyRelation
 
 from countess import VERSION
-from countess.core.parameters import NumericColumnChoiceParam, ColumnOrNoneChoiceParam
+from countess.core.parameters import ColumnOrNoneChoiceParam, NumericColumnChoiceParam
 from countess.core.plugins import DuckdbSimplePlugin
 from countess.utils.duckdb import duckdb_escape_identifier
 

countess/plugins/score_scale.py

@@ -8,9 +8,9 @@
     ArrayParam,
     ChoiceParam,
     ColumnChoiceParam,
-    NumericColumnChoiceParam,
     ColumnOrNoneChoiceParam,
     FloatParam,
+    NumericColumnChoiceParam,
     StringParam,
     TabularMultiParam,
 )

tests/test_parameters.py

@@ -330,7 +330,7 @@ def test_pcap():
     pp = IntegerParam("x")
     ap = PerColumnArrayParam("y", param=pp)
 
-    ap.set_column_choices({"a": True, "b": True, "c": True })
+    ap.set_column_choices({"a": True, "b": True, "c": True})
     assert len(ap) == 3
     apa, apb, apc = list(ap)