Closed LilyAnderssonLee closed 4 months ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit fce5e6f
+| ✅ 261 tests passed |+
!| ❗ 2 tests had warnings |!
@LilyAnderssonLee I think you were a bit overboard.
The idea is to be able to visualize quickly text file and html reports (cf https://github.com/nf-core/rnaseq/blob/dev/tower.yml). I would remove zip and gzip files.
cf https://docs.seqera.io/platform/23.4.0/reports/overview#limitations
@LilyAnderssonLee I think you were a bit overboard.
The idea is to be able to visualize quickly text file and html reports (cf https://github.com/nf-core/rnaseq/blob/dev/tower.yml). I would remove zip and gzip files.
cf https://docs.seqera.io/platform/23.4.0/reports/overview#limitations
Ok, I will update it.
@LilyAnderssonLee do you know how fancythe TSV viewer is?
If you can do filtering of tables indeed makes sense to include the taxonomic profiles, but if not I'm not sure the utility of inmcluding those.
HTML reports makes sense, asn does maybe log files, but otherwise...?
@jfy133 TBH I am not sure about the TSV viewer. I have never run taxprofiler on tower yet. It seems tower reports only support these formats: HTML, csv, tsv, pdf, and txt.
Apparently it does! (using nf-core/rnaseq on the AWS megatest workspace as an example!)
I have a feeling some of the tables will fail to render properly as extension is .txt
but Mimetype is not set to tSV
But I think we can review this and fix it in the future release, once we have a fulltest run to inspect the output files :)
Add output files information to the file
tower.yml
.Close https://github.com/nf-core/taxprofiler/issues/373
PR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).nextflow run . -profile debug,test,docker --outdir <OUTDIR>
).docs/usage.md
is updated.docs/output.md
is updated.CHANGELOG.md
is updated.README.md
is updated (including new tool citations and authors/contributors).