Skip to content

Commit

Permalink
Remove custom interpreter for sbatch
Browse files Browse the repository at this point in the history
Making the assumption that sbatch submission scripts are written in
bash. Too complicated to make them work for multiple shells.
  • Loading branch information
TimothyWillard committed Nov 12, 2024
1 parent b6c95ca commit 3fb4dbe
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion flepimop/gempyor_pkg/src/gempyor/batch.py
Original file line number Diff line number Diff line change
Expand Up @@ -660,7 +660,6 @@ def _sbatch_template(
Raises:
ValueError: If 'options' is found in `template_data` and `options` is not empty.
"""
template_data["interpreter"] = template_data.get("interpreter", "bash")
if "options" in template_data and options:
raise ValueError(
"Found 'options' in `template_data` but `options` is not empty, can only one."
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#!/usr/bin/env {{ interpreter }}
#!/usr/bin/env bash
{%- for opt in options %}
#SBATCH {{ opt }}
{%- endfor %}
Expand Down

0 comments on commit 3fb4dbe

Please sign in to comment.