Skip to content

Commit

Permalink
address comments
Browse files Browse the repository at this point in the history
  • Loading branch information
Michaelvll committed May 22, 2024
1 parent b996cf9 commit 6bf68fc
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions sky/utils/command_runner.py
Original file line number Diff line number Diff line change
Expand Up @@ -184,9 +184,8 @@ def _get_command_to_run(
# cluster by 1 second.
# sourcing ~/.bashrc is not required for internal executions
command += [
shlex.quote(
'true && export OMP_NUM_THREADS=1 PYTHONWARNINGS=ignore'
f' && ({cmd})')
shlex.quote('true && export OMP_NUM_THREADS=1 '
f'PYTHONWARNINGS=ignore && ({cmd})')
]
if not separate_stderr:
command.append('2>&1')
Expand Down Expand Up @@ -433,8 +432,7 @@ def run(
process_stream,
separate_stderr,
# A hack to remove the following SSH warning+bash warnings (twice):
# Warning: Permanently added 'xx.xx.xx.xx' to the list of known
# hosts.
# Warning: Permanently added 'xx.xx.xx.xx' to the list of known...
# bash: cannot set terminal process group
# bash: no job control in this shell
# When not source_bashrc, the bash warning will only show once.
Expand Down

0 comments on commit 6bf68fc

Please sign in to comment.