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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
singularity support #844
singularity support #844
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
might have to change some run
directives to script
s, especially for fastq downloading. Either way, better environments is nice!
@@ -1,6 +1,7 @@ | |||
""" | |||
all logic not related to any specific workflows should be here. | |||
""" | |||
container: "docker://quay.io/biocontainers/seq2science:0.7.2--pypyhdfd78af_0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this won't break anything right?
@@ -1,4 +1,4 @@ | |||
#!/usr/bin/perl | |||
#!/usr/bin/env perl |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
might need to add/change shebangs to other scripts too
What problem is the PR solving / What's new?
Let's maybe merge this just so I can do more testing with singularity 馃槆 (after a new release we get an automatic singularity container made for us), we won't add any documentation abuot singularity (yet). A nice addition is that it improves some envs of which not all requirements were properly defined.
Support for singularity should be relatively easy with this:
https://snakemake.readthedocs.io/en/stable/snakefiles/deployment.html#ad-hoc-combination-of-conda-package-management-with-containers
related: snakemake/snakemake#1588
Still needs plenty of streamlining, especially to be able to run seq2science itself from singularity, to test currently:
export SINGULARITY_BIND="/scratch/${USER},${CONDA_PREFIX},${TMPDIR},${XDG_CACHE_HOME}"
seq2science run rna-seq --cores 20 --snakemakeOptions use_singularity=True conda_prefix=/scratch/sande/singularity_test/envs
Checklist