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

Fix Tiara publishing files #520

Merged
merged 4 commits into from
Oct 4, 2023
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- [#514](https://github.com/nf-core/mag/pull/514) - Fix missing CONCOCT files in downstream output (reported by @maxibor, fix by @jfy133)
- [#515](https://github.com/nf-core/mag/pull/515) - Fix overwriting of GUNC output directories when running with domain classification (reported by @maxibor, fix by @jfy133)
- [#516](https://github.com/nf-core/mag/pull/516) - Fix edge-case bug where MEGAHIT re-uses previous work directory on resume and fails (reported by @husensofteng, fix by @prototaxites)
- [#517](https://github.com/nf-core/mag/pull/517) - Fix broken publishing of Tiara output files (fix by @jfy133)
jfy133 marked this conversation as resolved.
Show resolved Hide resolved

### `Dependencies`

Expand Down
13 changes: 3 additions & 10 deletions conf/modules.config
Original file line number Diff line number Diff line change
Expand Up @@ -698,17 +698,10 @@ process {

withName: TIARA_TIARA {
publishDir = [
[
path: { "${params.outdir}/Taxonomy/Tiara" },
path: { "${params.outdir}/Taxonomy/Tiara/" },
mode: params.publish_dir_mode,
pattern: { "${meta.assembler}-${meta.id}.tiara.{txt}" }
],
[
path: { "${params.outdir}/Taxonomy/Tiara/log" },
mode: params.publish_dir_mode,
pattern: { "log_${meta.assembler}-${meta.id}.tiara.{txt}" }
pattern: "*.txt"
]
]
ext.args = { "--min_len ${params.tiara_min_length} --probabilities" }
ext.prefix = { "${meta.assembler}-${meta.id}.tiara" }
}
Expand All @@ -720,7 +713,7 @@ process {

withName: TIARA_SUMMARY {
publishDir = [
path: { "${params.outdir}/Taxonomy/" },
path: { "${params.outdir}/GenomeBinning/Tiara" },
mode: params.publish_dir_mode,
pattern: "tiara_summary.tsv"
]
Expand Down
4 changes: 2 additions & 2 deletions docs/output.md
Original file line number Diff line number Diff line change
Expand Up @@ -434,12 +434,12 @@ Tiara is a contig classifier that identifies the domain (prokarya, eukarya) of c

- `Taxonomy/Tiara/`
- `[assembler]-[sample/group].tiara.txt` - Tiara output classifications (with probabilities) for all contigs within the specified sample/group assembly
- `log/log_[assembler]-[sample/group].txt` - log file detailing the parameters used by the Tiara model for contig classification.
- `log_[assembler]-[sample/group].txt` - log file detailing the parameters used by the Tiara model for contig classification.
- `GenomeBinning/tiara_summary.tsv` - Summary of Tiara domain classification for all bins.

</details>

Typically, you would use `tiara_summary.tsv` as the primary file to see which bins or unbins have been classified to which domains at a glance, whereas `[assembler]-[sample/group].tiara.txt` provides classifications for each contig.
Typically, you would use `tiara_summary.tsv` as the primary file to see which bins or unbins have been classified to which domains at a glance, whereas the files in `Taxonomy/Tiara` provides classifications for each contig.
jfy133 marked this conversation as resolved.
Show resolved Hide resolved
jfy133 marked this conversation as resolved.
Show resolved Hide resolved

### Bin sequencing depth

Expand Down
Loading