Skip to content
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

Missing "unzip" in the silva classifier step #35

Closed
d4straub opened this issue Nov 15, 2018 · 9 comments
Closed

Missing "unzip" in the silva classifier step #35

d4straub opened this issue Nov 15, 2018 · 9 comments
Assignees
Labels
bug Something isn't working

Comments

@d4straub
Copy link
Collaborator

d4straub commented Nov 15, 2018

Process make_SILVA_132_16S_classifier slipped our test runs.
Test command for fast processing e.g.:
nextflow run rrna-ampliseq -profile test,singularity --classifier false --dereplication 90

from #33

@d4straub d4straub self-assigned this Nov 15, 2018
@d4straub d4straub added this to the V1.1 Olive Steel Panda milestone Nov 15, 2018
@d4straub d4straub added the bug Something isn't working label Nov 15, 2018
@d4straub
Copy link
Collaborator Author

removed obvious problems but unzip still doesnt work

@apeltzer
Copy link
Member

That just requires to add conda-forge::unzip=6.0 to our environment.yaml and we're all good :-)

@d4straub
Copy link
Collaborator Author

works when I use my QIIME2 conda environment, but doesnt work with --profile singularity:
.command.sh: line 2: unzip: command not found

@apeltzer
Copy link
Member

Probably outdated container -it doesn#t download the updated one from yesterday if there is one present in your work/singularity/... folder :-)

@d4straub
Copy link
Collaborator Author

Locally with recently pulled github repo, deleted folders work and .nextflow and run

nextflow run rrna-ampliseq -profile test,singularity --classifier false --dereplication 90 --skip_fastqc --skip_ancom --skip_diversity_indices --skip_alpha_rarefaction

but still ".command.sh: line 2: unzip: command not found"

@apeltzer
Copy link
Member

Super weird, restarted the docker cloud build.

@apeltzer
Copy link
Member

Ah, I see - pipeline uses latest, change is only present on dev.
I'm pushing to master to force a container update here as well and then we're good :-)

@apeltzer
Copy link
Member

This fixes the issue locally - test it on the cluster and let me know how it goes :-)

@apeltzer apeltzer changed the title Fix make_SILVA_132_16S_classifier Missing "unzip" in the silva classifier step Nov 16, 2018
@apeltzer
Copy link
Member

Build finished, testing works and unzip is found :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants