-
Notifications
You must be signed in to change notification settings - Fork 21
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
Add kraken2 phylogenetic assignment subworkflow #47
base: dev
Are you sure you want to change the base?
Changes from 39 commits
036daf0
e7c1e71
4549e1a
c158510
d058ccf
e942664
033df6e
c5edd67
6c55393
59c4108
bf8eaa2
a9aba3e
07bd918
f4dc19a
d430a27
3e0d357
b6c0d88
ff8efdd
5b602dd
7b2f665
ca597f7
400a76b
4906699
54ce903
a3ef27b
4b7aaad
0192830
a6f5c6b
d8c31ec
7e41b19
82c68e7
3e9b674
dbb92c1
0324ead
f1759e1
dd9e07c
828abd6
31c1f82
3a22a9e
5a4ce9c
34b766d
b698d41
6dd4859
07dee47
abea138
a60d573
2488887
e28397a
9d8f631
5d1a15b
4729ada
bb8e3b0
758ce12
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -26,6 +26,31 @@ process { | |
ext.args = '--quiet' | ||
} | ||
|
||
withName: 'KRAKEN2_KRAKEN2' { | ||
publishDir = [ | ||
path: { "${params.outdir}/kraken2_reports" }, | ||
mode: params.publish_dir_mode, | ||
saveAs: { filename -> filename.equals('versions.yml') ? null : filename } | ||
] | ||
} | ||
|
||
withName: 'KRONA_KTIMPORTTAXONOMY' { | ||
publishDir = [ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same as before |
||
path: { "${params.outdir}/kraken2_reports/krona_reports" }, | ||
mode: params.publish_dir_mode, | ||
saveAs: { filename -> filename.equals('versions.yml') ? null : filename } | ||
] | ||
} | ||
|
||
withName: 'UNTAR' { | ||
publishDir = [ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not sure you want to output the kraken db, because it's size can be huge (depending on the selected one) and also it has been previously downloaded by the user, so already in user's device? You may want to use the storeDir in case you want to store the db and reuse it for later without the need of publishing it in the output There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I see what you mean! I could create a patch to the UNTAR module to add the storeDir directive, that would also need some changes to the config but it can be done. In any case, to avoid unnecessary waste of space by saving the uncompressed database, the pipeline works differently if the user provides a gzipped database or an uncompressed one. If the pipeline is gzipped, the The outputting of the uncompressed kraken2 db is turned off by default by the If the database is uncompressed, and the user passes a path to the |
||
path: { "${params.outdir}/kraken2_db" }, | ||
mode: params.publish_dir_mode, | ||
saveAs: { filename -> filename.equals('versions.yml') ? null : filename }, | ||
enabled: params.save_uncompressed_k2db | ||
] | ||
} | ||
|
||
withName: 'MULTIQC_GLOBAL' { | ||
ext.args = { params.multiqc_title ? "--title \"$params.multiqc_title\"" : '' } | ||
publishDir = [ | ||
|
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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.
I think there is a general statement for this in this (file)[https://github.com/nf-core/seqinspector/blob/31c1f829d97c4b98d21b68beed4af050fd331a37/conf/modules.config#L15], so I don't think is needed to add it twice, except if that bit is going to be removed it later?
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.
There is! I just added it here for two reasons: the first is that I wanted more descriptive names for the folders (
kraken2_reports
instead of justkraken2
) and I wanted the krona plots to be inside thekraken2_reports
folder, with a more descriptive name as well.The second reason I have added this seemingly redundant code is that kraken2 and kronatools can produce more output than what is produced now. I left these lines here looking into the future: they might need to be modified depending on the needs of the pipeline once it reaches a more stable status.