Closed d4straub closed 8 months ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit 7f3e4b6
+| ✅ 191 tests passed |+
#| ❔ 6 tests were ignored |#
!| ❗ 1 tests had warnings |!
Ah, almost everything is working again! What a journey!
test_iontorrent
& test_pacbio_its
both fail with
Process `NFCORE_AMPLISEQ:AMPLISEQ:DADA2_TAXONOMY_WF:DADA2_TAXONOMY (ASV_ITS_seqs.full.fasta,assignTaxonomy.fna)` terminated with an error exit status (137)
indicating that memory requirements are too high. This is curious because it passed in the past.
Both tests use dada_ref_taxonomy = "unite-fungi=8.3"
. unite-fungi=8.3
leads to a file with 18.8MB, 8.2
to 14.5MB, so testing the smaller one now.
edit: nope, doesnt work! Still
Command exit status:
137
edit2: running the test locally returns a memory usage of 6.045Gb, which is slightly over the limit of 6GB...
edit3: using sintax requires much less memory, so switching to sintax...
edit4: this requires unfortunately to disable addsh
testing
Thanks you two for your reviews, will merge it after tests passed.
Closes https://github.com/nf-core/ampliseq/issues/712
I had to change
test_iontorrent
&test_pacbio_its
to not use dada2 with unite but sintax (were cancelled because of too high memory requirements). That meant unfortunately disabling addsh which is now not tested anywhere, I think.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).