Snakemake-Profiles / slurm

Cookiecutter for snakemake slurm profile
MIT License
123 stars 44 forks source link

use explicit username instead of --me #101

Closed esrice closed 1 year ago

esrice commented 2 years ago

older versions of squeue (e.g., 17.02.11) do not support the "--me" option, so for backwards compatibility, using the "--user" flag followed by the username of the current user is helpful. The version of SLURM on my university's cluster is apparently several versions behind.

maarten-k commented 2 years ago

Sound like a good idea. Please change also the reference to this code in the comment of the code.

https://github.com/Snakemake-Profiles/slurm/blob/dec0b2e9f29320f9bfdbcf3546d8dd65e0cff95d/%7B%7Bcookiecutter.profile_name%7D%7D/slurm-sidecar.py#L4

https://github.com/Snakemake-Profiles/slurm/blob/dec0b2e9f29320f9bfdbcf3546d8dd65e0cff95d/%7B%7Bcookiecutter.profile_name%7D%7D/slurm-sidecar.py#L289

esrice commented 2 years ago

Thanks, how about now?

percyfal commented 1 year ago

@esrice LGTM, merging!