hiphase: stub output names differ from the real run (haplotagged BAMs and report files)
I maintainer di solito rispondono entro 1 giorno
Una pull request collegata è già stata integrata.
- #13431 di @aksenia — integrata
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 78/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Ambito
- bioinformatics, testing
Direzione di ricerca
Inizia dal blocco stub in modules/nf-core/hiphase/main.nf e confronta i nomi dei file che modifica con gli output effettivi dei comandi e i glob descritti nell’issue. Poi esamina i casi nf-test e lo snapshot di hiphase; il lavoro è completato quando le esecuzioni stub producono gli stessi nomi BAM e degli indici e i quattro nomi dei file dei report, un test copre due BAM di input e meta.yml documenta la convenzione di denominazione dei BAM.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Have you checked the docs?
Description of the bug
The HIPHASE stub writes different file names than the real run, so -stub output does not represent real output:
-
Haplotagged BAMs. The script uses
--output-bam ${file.baseName}_haplotagged.bam, but the stub touches${prefix}_${file.baseName}_haplotagged.bam(same for the.bai/.csiindex). Visible in the committed snapshot:- real:
NA03697B2_downsampled.pbmm2.repeats_haplotagged.bam - stub:
test_NA03697B2_downsampled.pbmm2.repeats_haplotagged.bam
A pipeline that pairs output BAMs back to samples by name gets nothing under
-stub. In genomic-medicine-sweden/nallo#1295 this let stub tests pass with an empty channel while real runs had a sample mix-up. - real:
-
Report files. The script writes
${prefix}.summary.${file_format},${prefix}.blocks.${file_format},${prefix}.stats.${file_format}and${prefix}.haplotag.${file_format}, but the stub touches${prefix}.${file_format}for all four, which matches none of the output globs (*.summary.tsvetc.). Thesummary_*,blocks_*,stats_*andhaplotag_*channels are empty in thetrue - tsv - stubandtrue - csv - stubsnapshots, while the corresponding real tests emit them.
Proposed fix:
- Stub: touch
${file.baseName}_haplotagged.bamand index (no prefix), and${prefix}.summary.${file_format}etc., matching the script. - Add a stub test with two input BAMs so the snapshot shows one output BAM per input with the expected names.
- Document in
meta.ymlthat output BAMs are named<input BAM basename>_haplotagged.bam, so callers can map outputs back to inputs by exact name.
Command used and terminal output
$ nf-test test modules/nf-core/hiphase/tests/main.nf.test --profile docker
No error; the mismatch is visible by comparing the real and stub entries in modules/nf-core/hiphase/tests/main.nf.test.snap (bams names differ; stub report-file channels are empty).
Relevant files
modules/nf-core/hiphase/main.nf(stub block)modules/nf-core/hiphase/tests/main.nf.test.snap- Downstream report: https://github.com/genomic-medicine-sweden/nallo/issues/1295
System information
Not environment-specific; found from the module code and committed snapshot at 32d9d7b (current master).
- Lingua principale
- Nextflow
- Stelle
- 433
- Fork
- 1.1k
- Merge medio
- 9h 12m
- PR unite (30g)
- 85
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di nf-core/modules
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
nf-core/modules#13434 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
star/genomegenerate: --limitGenomeGenerateRAM leaves only 100 MB below task memory, so the container limit can kill STARForse già presa @danilomonge l’ha presa 6 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
I maintainer di solito rispondono entro 1 giorno
-
update module: KRAKEN2/* 2.1.6 -> 2.17.2Forse già presa @barlowa124 l’ha presa 16 giorni fa. Apertaupdate module
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
nf-core/modules#13078 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
jellyfish/count crashes on paired-end inputForse già presa @barlowa124 l’ha presa 19 giorni fa. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di nf-core/modules
Issue simili
-
convert: no range check between Phred encodings, so a wrong --from writes a broken FASTQ (exit 0)Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
shenwei356/seqkit#612 · 1 commento ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
bioconda/bioconda-recipes#70400 ·
I maintainer di solito rispondono entro 2 giorni
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
LCrossman/microBioRust#57 ·
-
genomics tests: mdata_factor_analysis fails to import under mudata 0.4.x (set_options removed)Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
shackett/shackett-utils#18 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100