-
Notifications
You must be signed in to change notification settings - Fork 16
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
Re-structure pipeline #134
Conversation
Adjustment of the voting scheme and updated viral taxonomy and bugfixes for new release
Fix Chromomap container
Merge small set of fixes for new release #123
subworkflows/local/annotate.nf
Outdated
viphos_annotations = ANNOTATION.out.map { _, __, annotations -> annotations }.collect() | ||
taxonomy_annotations = ASSIGN.out.map { _, __, taxonomy -> taxonomy }.collect() | ||
checkv_results = CHECKV.out.map { _, __, quality_summary, ___ -> quality_summary }.collect() | ||
viphos_annotations = ANNOTATION.out.annotations.groupTuple().map{ |
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.
Is this groupTuple
required becase ANNOTATION
return a list of files? I'm not sure I follow
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.
it is in development stage yet, I'm not sure what it returns either :D
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.
Perfect hehe
So far.. it look goods to me :) Thank you @KateSakharova |
cd0039a
to
74d296b
Compare
Restructured pipeline to nf-core view
Now virify supports samplesheet (fq1 and fq2 are optional)
id,assembly,fq1,fq2
For assembly as input:
If you have assembly as input you can use old argument --fasta - that will run pipieline for 1 assembly.
If you have more than 1 assembly - use samplesheet.
For raw_reads as input:
You need to use ONLY samplesheet (even for 1 run). I replaced --illumina option with --assemble (boolean). User should specify that flag to force assembly process (that flag will help user to understand what he is doing). Hopefully, nobody uses it because it is experimental feature.