Please add a clear description of what the PR is about:
An user ran into error due to snakmake's sbatch command args being hardcoded. This PR refactors to remove this hardcoding and instead uses args and values from cluster config file supplied via --cluster_config. Cluster config is now added as under requirement in docs.
Please fill in the checklist below and comment as needed.
[x] Was code modified? Briefly describe. Yes. See above description
[x] Was documentation modified? Briefly describe. Yes. See above description
[x] Is this a bug-fix? Briefly describe. Yes. See above description
[x] Is this a feature addition? Briefly describe. No
[x] Did you modify QuaC-Watch config file? If so, did you modify multiqc template
configs/multiqc_config_template.jinja2 and script src/quac_watch/create_mutliqc_configs.py as necessary? No
[x] Did you perform system-level testing manually as described in master readme doc? Did it pass completely? If not why? Yes.
[x] Updated Changelog.md file with change logs in recommended format?
Anything else reviewer should know?
Also updates PR template to add a description section.
Pull request
Please add a clear description of what the PR is about:
An user ran into error due to snakmake's
sbatch
command args being hardcoded. This PR refactors to remove this hardcoding and instead uses args and values from cluster config file supplied via--cluster_config
. Cluster config is now added as under requirement in docs.Please fill in the checklist below and comment as needed.
configs/multiqc_config_template.jinja2
and scriptsrc/quac_watch/create_mutliqc_configs.py
as necessary? NoChangelog.md
file with change logs in recommended format?Anything else reviewer should know?
Also updates PR template to add a description section.