Closed gennadyFauna closed 3 months ago
nf-core lint
overall result: Passed :white_check_mark: :warning:Posted for pipeline commit 600619f
+| ✅ 171 tests passed |+
#| ❔ 4 tests were ignored |#
!| ❗ 3 tests had warnings |!
I am working on testing the current version to have a baseline. There is a potential existing bug: using the kallisto
pseudoaligner with the lamanno
workflow works, but only if Nextflow builds a new reference. Specifying an existing reference using kallisto_gene_map
(the t2g.txt
file?) and kallisto_index
(the kb_ref_out.idx
file) does not work, and throws an error pointing at
https://github.com/nf-core/scrnaseq/blob/90cb6a48155248286c85395c53a201c3a31b2258/workflows/scrnaseq.nf#L147 with the following explanation:
Missing process or function mix([DataflowStream[?], [DataflowGetPropertyExpression(value=null)]])
I suspect that this is because KALLISTOBUSTOOLS_COUNT
takes in the cDNA/intron FASTA files in this line:
https://github.com/nf-core/scrnaseq/blob/90cb6a48155248286c85395c53a201c3a31b2258/subworkflows/local/kallisto_bustools.nf#L59
But I do not believe these variables (t1c
and t2c
) are ever defined if the index is provided:
https://github.com/nf-core/scrnaseq/blob/90cb6a48155248286c85395c53a201c3a31b2258/subworkflows/local/kallisto_bustools.nf#L43
I assume fixing this bug would require changing the kallisto
workflow parameters. Though it does look as though these are already present in the following line:
https://github.com/nf-core/scrnaseq/blob/90cb6a48155248286c85395c53a201c3a31b2258/modules/nf-core/kallistobustools/count/meta.yml#L35
Fixed some issues in kallisto_bustools.nf
: the way the code was set up made it impossible to pass in an existing kallisto reference. Removed t2g.py
. This script is from 2017 and a few versions behind; it may be more reliable to just use the t2g.txt
generated by kallisto.
In the test run, kb exits with a SIGKILL error: https://github.com/nf-core/scrnaseq/actions/runs/7876828275/job/21493304177#step:5:93. This is actually likely related to the binary issue in https://github.com/pachterlab/kallisto/issues/411.
@nf-core-bot fix linting
Don't worry about the linting failures. This is related to an update in the nf-core template that has not been yet integrated into the scrnaseq pipeline.
@nf-core-bot fix linting
Starting work on bumping up kb version to 0.28.2. The
nucleus
andlamanno
workflows are obsolete as of 0.28.0, so the interface for generating count matrices needs to be updated.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).