Apply some low hanging lints

Author: felixhekhorn

src/pineko/cli/gen_sv.py

@@ -3,7 +3,6 @@
 import pathlib
 
 import click
-import pineappl
 import rich
 
 from .. import scale_variations

src/pineko/cli/kfactor.py

@@ -3,7 +3,6 @@
 import pathlib
 
 import click
-import rich
 
 from .. import kfactor
 from ._base import command

src/pineko/comparator.py

@@ -30,7 +30,7 @@ def compare(pine, fktable, max_as, max_al, pdf, xir, xif):
     df : pd.DataFrame
         comparison table
     """
-    import lhapdf  # pylint: disable=import-error
+    import lhapdf  # pylint: disable=import-error,import-outside-toplevel
 
     pdfset = lhapdf.mkPDF(pdf, 0)
     pdgid = int(pdfset.set().get_entry("Particle"))

src/pineko/kfactor.py

@@ -308,10 +308,10 @@ def do_it(
         (first_nonzero_order[0] + max_as_test, min_al, 0, 0), grid_orders_filtered
     )
     if is_in and not order_exists:
-        rich.print(f"[green] Success: Requested order already in the grid.")
+        rich.print("[green] Success: Requested order already in the grid.")
         return
-    elif not is_in and order_exists:
-        rich.print(f"[red] Abort: order exists is True but order not in the grid.")
+    if not is_in and order_exists:
+        rich.print("[red] Abort: order exists is True but order not in the grid.")
         return
     construct_and_merge_grids(
         grid_path,
@@ -329,19 +329,19 @@ def filter_k_factors(pigrid, centrals_kfactor):
     """Filter the centrals k-factors according to their lenght compared to the number of bins of the grid."""
     centrals_kfactor_filtered = np.array([])
     if pigrid.bins() == len(centrals_kfactor):
-        rich.print(f"[orange] The number of bins match the lenght of the k-factor.")
+        rich.print("[orange] The number of bins match the lenght of the k-factor.")
         centrals_kfactor_filtered = centrals_kfactor
     elif pigrid.bins() < len(centrals_kfactor):
         rich.print(
-            f"[yellow] The number of bins is less than the lenght of the k-factor."
+            "[yellow] The number of bins is less than the lenght of the k-factor."
         )
         if not all(elem == centrals_kfactor[0] for elem in centrals_kfactor):
             # This case is actually wrong.
             raise ValueError("KFactor contains too many different values.")
         centrals_kfactor_filtered = centrals_kfactor
     else:
         rich.print(
-            f"[yellow] The number of bins is more than the lenght of the k-factor."
+            "[yellow] The number of bins is more than the lenght of the k-factor."
         )
 
         # This is the last case in which grid.bins() > len(centrals_kfactor)
@@ -388,7 +388,7 @@ def compute_k_factor_grid(
     target_folder: pathlib.Path
         path where store the new grid (optional)
     """
-    import lhapdf  # pylint: disable=import-error
+    import lhapdf  # pylint: disable=import-error,import-outside-toplevel
 
     # With respect to the usual convention here max_as is max_as-1
     max_as_test = max_as - 1

src/pineko/scale_variations.py

@@ -5,7 +5,6 @@
 
 import numpy as np
 import pineappl
-import rich
 from eko import beta
 
 from . import check
@@ -18,12 +17,12 @@
 class ReturnState(Enum):
     """Auxiliary class to list the possible return states."""
 
-    ALREADY_THERE = f"[green]Renormalization scale variations are already in the grid"
+    ALREADY_THERE = "[green]Renormalization scale variations are already in the grid"
     ORDER_EXISTS_FAILURE = (
         "Order_exists is True but the order does not appear to be in the grid"
     )
     MISSING_CENTRAL = "Central order is not high enough to compute requested sv orders"
-    SUCCESS = f"[green]Success: scale variation orders included!"
+    SUCCESS = "[green]Success: scale variation orders included!"
 
 
 def qcd(order: OrderTuple) -> int:
@@ -168,7 +167,7 @@ def write_grids(gridpath, grid_list):
 
 
 def merge_grids(
-    gridpath, grid_list_path, target_path=None, nec_orders={}, order_exists=False
+    gridpath, grid_list_path, target_path=None, nec_orders=None, order_exists=False
 ):
     """Merge the single grids in the original."""
     grid = pineappl.grid.Grid.read(gridpath)
@@ -177,7 +176,9 @@ def merge_grids(
     else:
         target_path = target_path / gridpath.name
     if order_exists:
-        grid = construct_and_dump_order_exists_grid(grid, list(nec_orders.keys())[0])
+        grid = construct_and_dump_order_exists_grid(
+            grid, list(nec_orders.keys())[0] if nec_orders is not None else []
+        )
     for grid_path in grid_list_path:
         grid.merge_from_file(grid_path)
         grid_path.unlink()

src/pineko/theory.py

@@ -325,8 +325,7 @@ def eko(self, name, _grid, tcard):
             if not self.overwrite:
                 rich.print(f"Skipping existing operator {eko_filename}")
                 return
-            else:
-                eko_filename.unlink()
+            eko_filename.unlink()
         # do it!
         logger.info("Start computation of %s", name)
         start_time = time.perf_counter()