-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make the slurm_account
parameter optional, because it is in fact optional.
#76
Comments
Just taught a course on a cluster, with a default resp. None-account. Worked like a charm. So, I wonder, what is the output of |
Here is the output:
This pattern repeats for 702 lines. |
That is really weird. But I might be able to work with that under the assumption, that is “only” some sort of whitespace. |
Just copy-pasted your literal feedback - thank you! - into an editor. It indeed contains a hidden Now, I wonder: What is this #!/usr/bin/env python3
import os
import subprocess
cmd = f'sacct -nu "{os.environ["USER"]}" -o Account%256 | head -n1'
sacct_out = subprocess.check_output(
cmd, shell=True, text=True, stderr=subprocess.PIPE
)
with open('sacct.out', 'wb') as outfile:
outfile.write(sacct_out) PS What is the output of the |
The string The script you provided raises a TypeError, because the file is opened in bytes mode and not text mode. Also the |
My bad, I should have included If it does not work, we need to check more details. |
Thanks for looking into this. I tested this unmerged pull request like this:
And at the end:
The result is the following being printed:
So it worked perfectly 👌🏻 |
Some SLURM installations don't require specifying an account when using
sbatch
orsalloc
.Currently the value
-A (null)
is actually used in thesbatch
call if you don't mention one:And if you set it to some placeholder value such as "default" for instance, this happens:
The text was updated successfully, but these errors were encountered: