Very nice workflow! A small thing:
I would suggest modifying some of the slurm parameters:
--qos should not be mandatory as not every cluster needs it (ours doesnt)
it would be very helpful if there would be an option from the command line to access the Nextflow clusterOptions (https://www.nextflow.io/docs/latest/process.html#process-clusteroptions). We for example have a mandatory account parameter (-A) in our Slurm config, others might have other configs, so this would help to make the workflow more easily configurable if run on other HPC setups.
Kudos for the workflow, it is very well written and I was able to run it on our HPC after the clusterOption modification. If you want I can modify it and create a PR
Very nice workflow! A small thing:
I would suggest modifying some of the slurm parameters:
--qos should not be mandatory as not every cluster needs it (ours doesnt)
it would be very helpful if there would be an option from the command line to access the Nextflow clusterOptions (https://www.nextflow.io/docs/latest/process.html#process-clusteroptions). We for example have a mandatory account parameter (-A) in our Slurm config, others might have other configs, so this would help to make the workflow more easily configurable if run on other HPC setups.
Kudos for the workflow, it is very well written and I was able to run it on our HPC after the clusterOption modification. If you want I can modify it and create a PR