SLURM with multiple dependencies

evoldoers / biomake

GNU-Make-like utility for managing builds and complex workflows

BSD 3-Clause "New" or "Revised" License

100 stars 9 forks source link

SLURM with multiple dependencies #49

Closed sjackman closed 6 years ago

sjackman commented 6 years ago

There appears to be an errant space character before the comma in --dependency=afterok:165337 ,afterok:165339

❯❯❯ biomake -Q slurm
…
Submitting job: sbatch -o /home/sjackman/work/redcedar/.biomake/slurm/out/all -e /home/sjackman/work/redcedar/.biomake/slurm/err/all   -n16 --dependency=afterok:165337 ,afterok:165339  --parsable /home/sjackman/work/redcedar/.biomake/slurm/script/all >/home/sjackman/work/redcedar/.biomake/slurm/job/all
sbatch: error: Unable to open file ,afterok:165339

sjackman commented 6 years ago

@ihh Do you have time to look at this issue as well?

ihh commented 6 years ago

I will make tomorrow a biomake work day ;)

sjackman commented 6 years ago

Thanks, Ian! 😀 ❤️

sjackman commented 6 years ago

🎉 Thank you, Ian! I'm very excited to test out this fix.