ruff fixes

ubermagtable/table.py

@@ -698,10 +698,7 @@ def slider(self, x=None, multiplier=None, description=None, **kwargs):
         labels = np.around(values / multiplier, decimals=2)
         options = list(zip(labels, values))
 
-        if x == "t":
-            units = f" ({ubermagutil.units.rsi_prefixes[multiplier]}s)"
-        else:
-            units = ""
+        units = f" ({ubermagutil.units.rsi_prefixes[multiplier]}s)" if x == "t" else ""
         if description is None:
             description = f"{x}{units}:"
 

ubermagtable/tests/test_table.py

@@ -69,7 +69,7 @@ def test_init(self):
 
     def test_fromfile(self):
         for odtfile in self.odtfiles:
-            for rename in [True, False]:
+            for _rename in [True, False]:
                 table = ut.Table.fromfile(odtfile)
                 check_table(table)
 

ubermagtable/tests/test_util.py

@@ -41,7 +41,7 @@ def test_units():
             units = uu.units(odtfile, rename=rename)
 
             assert isinstance(units, dict)
-            assert all(isinstance(unit, str) for unit in units.keys())
+            assert all(isinstance(unit, str) for unit in units)
             assert all(isinstance(unit, str) for unit in units.values())
             assert "J" in units.values()  # Energy is always in
             assert "" in units.values()  # Columns with no units are always in

ubermagtable/util/util.py

@@ -135,10 +135,10 @@
 
 
 def rename_column(name, cols_dict):
-    if name in cols_dict.keys():
+    if name in cols_dict:
         return cols_dict[name]
 
-    for key in cols_dict.keys():
+    for key in cols_dict:
         if len(key.split("::")) == 2:
             start, end = key.split("::")
             name_split = name.split(":")