From 6e2e0dd58a8d8ffdd5cf802389ce15b1b417ca6d Mon Sep 17 00:00:00 2001 From: Harichandra Prasath <129069095+Harichandra-Prasath@users.noreply.github.com> Date: Mon, 18 Mar 2024 23:00:58 +0530 Subject: [PATCH] Remove permit_multiple_nodes argument in SimpleLauncher (#3242) --- parsl/launchers/launchers.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/parsl/launchers/launchers.py b/parsl/launchers/launchers.py index 4845284376..021357b56c 100644 --- a/parsl/launchers/launchers.py +++ b/parsl/launchers/launchers.py @@ -11,11 +11,13 @@ class SimpleLauncher(Launcher): def __init__(self, debug: bool = True) -> None: super().__init__(debug=debug) - def __call__(self, command: str, tasks_per_node: int, nodes_per_block: int, permit_multiple_nodes: bool = False) -> str: + def __call__(self, command: str, tasks_per_node: int, nodes_per_block: int) -> str: + + if nodes_per_block > 1: + logger.warning('Simple Launcher only supports single node per block. ' + f'Requested nodes: {nodes_per_block}. ' + 'You may be getting fewer workers than expected') - if nodes_per_block > 1 and not permit_multiple_nodes: - logger.warning("SimpleLauncher only supports 1 node per block. " - "Set permit_multiple_nodes=True to allow multiple nodes per block.") return command