MCMICRO reader does not support new nf-core/mcmicro output formats
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 48/100
Línea de trabajo
Comienza en spatialdata_io.readers.mcmicro y compara sus suposiciones con los dos diseños de salida enumerados en el issue. Rastrea cómo se descubren el registro, la segmentación, la cuantificación, los marcadores y los núcleos TMA, y define casos de validación para múltiples muestras y segmentadores, la detección automática de la pipeline y la sobrescritura explícita. Se considera terminado cuando las salidas tanto de labsyspharm como de nf-core siguen siendo legibles, incluidos los casos de WSI y TMA.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
We recently completed a port of MCMICRO to nf-core (https://nf-co.re/mcmicro/2.0.0). There are minor changes to expected output formats, and the current spatialdata_io.readers.mcmicro reader — written against the original labsyspharm/mcmicro layout — does not read nf-core output.
What changed between the two pipelines:
| Aspect | labsyspharm/mcmicro | nf-core/mcmicro |
|---|---|---|
| Run config | qc/params.yml (workflow.tma) |
pipeline_info/params*.json (tma_dearray, segmentation) |
| Registration | registration/<sample>.ome.tif |
registration/ashlar/<sample>.ome.tif |
| Segmentation | segmentation/<module>-<sample>/{cell,nuclei}.ome.tif |
segmentation/<segmenter>/<sample>_<tool-suffix>.tif |
| Quantification | quantification/<module>-<sample>_<comp>.csv |
quantification/mcquant/<segmenter>/<sample>.csv |
| TMA cores | dearray/ + qc/coreograph/centroidsY-X.txt |
tma_dearray/ (plus TMA_MAP.tif) + tma_dearray/centroidsY-X.txt |
| Markers | markers.csv at root |
input sheet (often not copied to outdir); backsub/<sample>_backsub.csv when backsub runs |
| Samples / segmenters | single sample, typically one segmenter | multiple samples and multiple segmenters per run |
Gotchas found while testing against real nf-core/mcmicro output that will need to be considered:
- Segmentation mask filenames are tool-specific, not a uniform
_masksuffix — e.g.1_cp_masks.tif(cellpose) vsexemplar-002_1_mask.tif(mesmer). - The segmenter directory name differs between
segmentation/andquantification/mcquant/(e.g. deepcell_mesmer vs mesmer), so tables can't be linked to labels by an exact directory-name match. - With background subtraction, the registration image and the quantification tables carry different marker sets (e.g. 40 image channels vs 37 table markers).
- TMA core numbers must be parsed carefully — the sample name itself can contain digits (
exemplar-002_1_mask→ core1, not002) — andtma_dearray/also containsTMA_MAP.tif, which is not a core. - The
prelude/markers_markersheet_mqc.tsvfile is a long-format MultiQC validation report, not a usable wide marker sheet.
Proposal
Update the mcmicro reader to auto-detect the pipeline (with an explicit pipeline= override) and handle both layouts, including multiple samples and segmenters, in WSI and TMA modes. Backwards compatibility with labsyspharm output is preserved.
- Lenguaje dominante
- Python
- Estrellas
- 103
- Forks
- 65
- Merge medio
- 1 h 8 min
- PR fusionados (30 d)
- 3
Preparar el entorno
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 scverse/spatialdata-io
-
documentation
Dificultad 2/5 Medio día Aptitud para principiantes 65/100
scverse/spatialdata-io#161 · 13 comentarios ·
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 58/100
scverse/spatialdata-io#422 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
scverse/spatialdata-io#419 · 1 comentario ·
-
Visium in matrix market formatAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
scverse/spatialdata-io#408 ·
-
xenium
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
scverse/spatialdata-io#407 ·
Todos los issues de scverse/spatialdata-io
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
https://search.utilibre.orgAbiertoinstance instance add
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
searxng/searx-instances#941 · 1 comentario ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
FluidNumerics/fluid-walk-blocker#89 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día