linter

CrayLabs · Aug 3, 2023 · 45df9d1 · 45df9d1
1 parent f996d4a
commit 45df9d1
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/smartsim/_core/launcher/step/alpsStep.py b/smartsim/_core/launcher/step/alpsStep.py
@@ -32,7 +32,7 @@
 from ....error import AllocationError
 from ....log import get_logger
 from .step import Step
-from ....settings import AprunSettings, Singularity
+from ....settings import AprunSettings
 
 logger = get_logger(__name__)
 

diff --git a/smartsim/_core/launcher/step/lsfStep.py b/smartsim/_core/launcher/step/lsfStep.py
@@ -31,7 +31,7 @@
 from ....error import AllocationError
 from ....log import get_logger
 from .step import Step
-from ....settings import BsubBatchSettings, JsrunSettings
+from ....settings import BsubBatchSettings
 from ....settings.base import RunSettings
 
 logger = get_logger(__name__)
@@ -234,10 +234,11 @@ def _make_mpmd(self) -> None:
         """Build LSF's Explicit Resource File"""
         erf_file_path = self.get_step_file(ending=".mpmd")
 
+
         # Find launch_distribution command
         preamble_lines = self.run_settings.mpmd_preamble_lines.copy()
         distr_line = None
-        for line in self.run_settings.mpmd_preamble_lines:
+
             if line.lstrip(" ").startswith("launch_distribution"):
                 distr_line = line
                 preamble_lines.remove(line)

diff --git a/smartsim/_core/launcher/step/mpiStep.py b/smartsim/_core/launcher/step/mpiStep.py
@@ -142,7 +142,7 @@ def _make_mpmd(self) -> t.List[str]:
             cmd += mpmd.format_run_args()
             cmd += mpmd.format_env_vars()
             cmd += mpmd.exe
-            cmd += mpmd._exe_args
+            cmd += mpmd._exe_args  # pylint: disable=protected-access
 
         cmd = sh_split(" ".join(cmd))
         return cmd