-
Notifications
You must be signed in to change notification settings - Fork 24
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
Batch job submission failed: Node count specification invalid #97
Comments
Removing this outer At least on my university's slurm setup, slurm requires the There is no other mention of the |
Ah, yes - the code there does not make sense! The intent got lost! Thank you for spotting this, I will code a fix, tomorrow. |
Thanks, I was worried something was weird on our cluster! |
🤖 I have created a release *beep* *boop* --- ## [0.5.2](v0.5.1...v0.5.2) (2024-06-04) ### Bug Fixes * [#97](#97) preventing node confinment ([#98](#98)) ([fa7877f](fa7877f)) --- This PR was generated with [Release Please](/~https://github.com/googleapis/release-please). See [documentation](/~https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Na, things happen ... Thanks for your excellent report. A bug fix release is out. Please install and test, one the bioconda release is ready. |
The issue is fixed at least in version 0.6.0 on Bioconda. Sorry it took so long for me to test, and thanks for fixing! |
Using the following Snakefile:
And running: (where <act_name> is my account name)
I get:
I also replaced
sbatch
in my local bash path and used it toecho
out the arguments that are passed tosbatch
:I don't see any
nodes
argument...Versions:
Conda list:
The text was updated successfully, but these errors were encountered: