MCMICRO reader does not support new nf-core/mcmicro output formats
まだ誰も着手していません。
評価
調査の方向性
spatialdata_io.readers.mcmicro から始め、issue に記載されている 2 つの出力レイアウトと、その前提を比較します。registration、segmentation、quantification、markers、TMA cores がどのように検出されるかを追跡し、複数のサンプルと segmenters、pipeline の自動検出、明示的な override の検証ケースを定義します。完了の条件は、WSI と TMA のケースを含め、labsyspharm と nf-core の両方の出力を引き続き読み取れることです。
索引モデルが issue の本文から書いたものです。
説明
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.
- 主要言語
- Python
- スター
- 103
- フォーク
- 65
- 平均マージ
- 1時間 8分
- マージ済み PR（30日）
- 3
環境構築
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
scverse/spatialdata-io のほかの issue
-
documentation
難易度 2/5 半日 初心者へのやさしさ 65/100
scverse/spatialdata-io#161 · コメント 13 件 ·
-
bug
難易度 3/5 1〜2日 初心者へのやさしさ 58/100
scverse/spatialdata-io#422 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
scverse/spatialdata-io#419 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 52/100
scverse/spatialdata-io#408 ·
-
xenium
難易度 4/5 3〜5日 初心者へのやさしさ 52/100
scverse/spatialdata-io#407 ·
scverse/spatialdata-io の issue をすべて見る
似ている issue
-
correction metadata
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
acl-org/acl-anthology#10104 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug status/needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
prowler-cloud/prowler#12885 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Bug in GaussianTailProbabilityCalibrator: running_statistics=False still uses a windowed varianceオープンbug good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
selimfirat/pysad#107 ·
メンテナーはふだん 1 日以内に返信
-
bug ci-failure high priority
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
vllm-project/vllm-omni#8194 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信