Skip to content

Commit

Permalink
Remove the opportunity to configure script_dir for LocalChannel
Browse files Browse the repository at this point in the history
Prior to this PR, if the argument defaulted to None, then the DFK would
set a script dir based on the workflow rundir.

Post this PR, this default behaviour is the only behaviour and the user
cannot specify an overriding value.

Future PRs will remove channel level script directories entirely.
This is part of issues #3515 removing channels,
  • Loading branch information
benclifford committed Nov 7, 2024
1 parent 6683f5b commit 74dc193
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions parsl/channels/local/local.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,14 +15,14 @@ class LocalChannel(Channel, RepresentationMixin):
and done so infrequently that they do not need a persistent channel
'''

def __init__(self, script_dir=None):
def __init__(self):
''' Initialize the local channel. script_dir is required by set to a default.
KwArgs:
- script_dir (string): Directory to place scripts
'''
self.hostname = "localhost"
self.script_dir = script_dir
self.script_dir = None

def execute_wait(self, cmd, walltime=None):
''' Synchronously execute a commandline string on the shell.
Expand Down

0 comments on commit 74dc193

Please sign in to comment.