hiphase: stub output names differ from the real run (haplotagged BAMs and report files)
Los mantenedores suelen responder en 1 día
Ya se ha fusionado un pull request relacionado.
- #13431 de @aksenia — fusionado
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Área
- bioinformatics, testing
Línea de trabajo
Empieza por el bloque stub de modules/nf-core/hiphase/main.nf y compara los nombres de archivo que modifica con las salidas reales de los comandos y los globs descritos en el issue. Después, revisa los casos de nf-test y el snapshot de hiphase; el trabajo estará listo cuando las ejecuciones stub produzcan los mismos nombres de BAM e índices y los cuatro nombres de archivo de los informes, haya una prueba que cubra dos BAM de entrada y meta.yml documente la nomenclatura de los BAM.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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).
- Lenguaje dominante
- Nextflow
- Estrellas
- 433
- Forks
- 1.1k
- Merge medio
- 13 h 30 min
- PR fusionados (30 d)
- 222
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de nf-core/modules
-
update module: SAWFISH/JOINTCALLAbiertoupdate module
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
update module: SAWFISH/DISCOVERAbiertoupdate module
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
nf-core/modules#13434 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
star/genomegenerate: --limitGenomeGenerateRAM leaves only 100 MB below task memory, so the container limit can kill STARPosiblemente ocupada @danilomonge la tomó hace 9 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
Los mantenedores suelen responder en 1 día
Todos los issues de nf-core/modules
Issues similares
-
New term request
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
monarch-initiative/mondo#10796 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
-
convert: no range check between Phred encodings, so a wrong --from writes a broken FASTQ (exit 0)Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
shenwei356/seqkit#612 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
bioconda/bioconda-recipes#70400 ·
Los mantenedores suelen responder en 1 día
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
LCrossman/microBioRust#57 ·