new module: insitugame by Lukecele · Pull Request #13471 · nf-core/modules

Lukecele · 2026-10-27T12:53:14Z

Summary

This pull request introduces the new module insitugame to nf-core/modules.

insitugame is a computational oncology assay for inferring tumor subclonal payoff matrices, resolving Nash coexistence equilibria, and evaluating adaptive therapy regimens directly from intact spatial transcriptomics data (10x Genomics Visium, CytAssist, and spatial coordinates).

Verification & Compliance

Built using official nf-core modules create templating (nf-core/tools v4.1.0).
Linting: Passed all 59/59 tests with 0 warnings and 0 errors (nf-core modules lint insitugame).
Testing: Both regular and -stub test suites pass via nf-test test modules/nf-core/insitugame/tests/main.nf.test with recorded snapshots.
Software availability: Official Bioconda recipe merged and published (bioconda::insitugame=1.0.6). BioContainers image publication is pending (see comment below).

Checklist

This PR fixes an issue or adds a new module.
I have read the contribution guidelines.
I have updated the meta.yml and environment.yml with valid metadata and conda pin.

…hook

Lukecele · 2026-10-27T21:02:16Z

Hi @famosab the module works fine locally and Conda tests are passing perfectly here in CI (~50s). However, Docker and Singularity keep failing with Error 125 due to a container pull issue from GHCR. The GHCR container is set to public. Could you take a look or let me know if there is a workaround to bypass this network timeout and get it merged?"

Lukecele · 2026-10-27T22:41:00Z

Update: license header added to main.nf (per #13448). The remaining CI failures are not caused by this PR:

pre-commit fails on subworkflows/nf-core/bcl_demultiplex/main.nf ("Shebang comment should appear at the first line @ line 4"), a file this PR does not touch. This branch is up to date with master.
Docker/Singularity fail because the BioContainers image quay.io/biocontainers/insitugame:1.0.6--pyhdfd78af_0 is not published yet. The Bioconda recipe (Add insitugame v1.0.6 bioconda/bioconda-recipes#70794) was merged today and the upload job ran, but other recently merged new packages (e.g. naivede) are in the same state, while an older one (lrcorrect, merged yesterday) is available. I will re-run CI once the image is pullable.

Conda tests and module lint pass.

famosab · 2026-10-28T09:10:01Z

+        'https://depot.galaxyproject.org/singularity/insitugame:1.0.6--pyhdfd78af_0':
+        'quay.io/biocontainers/insitugame:1.0.6--pyhdfd78af_0' }"


We are using seqera containers for the modules now :) Can you change to that?

famosab · 2026-10-28T09:11:13Z

+    script:
+    def args = task.ext.args ?: ''
+    def prefix = task.ext.prefix ?: "${meta.id}"
+    """


You are the author of the tool right? Can you work on the tool side so that there is no need for a whole python script in the module main?

Lukecele · 2026-10-28T10:02:05Z

Hi @famosab, I can update the module to the Seqera/Wave container, but the package does not currently expose a CLI, so I cannot remove the inline Python without a new insitugame release. Could you provide the Wave-generated Docker image and Singularity/Apptainer digest for insitugame 1.0.6, or confirm the expected Wave package/container reference? I do not want to commit a guessed image tag. Once provided, I will update only main.nf and rerun the module tests.

famosab · 2026-10-28T10:15:06Z

Can you expose the CLI then? I think that is a better way of handling things.

You can generate the containers yourself here https://seqera.io/containers/

Lukecele · 2026-10-28T10:18:13Z

I' actually working on ultimate version for this... Patience and thank you Il mer 28 ott 2026, 11:15 Famke Bäuerle ***@***.***> ha scritto:

…

*famosab* left a comment (nf-core/modules#13471) <#13471 (comment)> Can you expose the CLI then? I think that is a better way of handling things. You can generate the containers yourself here https://seqera.io/containers/ — Reply to this email directly, view it on GitHub <#13471?email_source=notifications&email_token=AU3A7UX6KRWNHC6OBUMSLTL5WHBMBA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTMNBWGYZTEMJVGAY2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KYZTPN52GK4S7MNWGSY3L#issuecomment-6466321501>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AU3A7UQASGLFT2XTHAHY7NL5WHBMBAVCNFSNUABFKJSXA33TNF2G64TZHMYTSOBZHAZDAOBTHNEXG43VMU5TMMBVHE2TMMRZGEZKC5QC> . Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS <https://github.com/notifications/mobile/ios/AU3A7UUHHHUWT5EWZ4M6YTD5WHBMBA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTMNBWGYZTEMJVGAY2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KUZTPN52GK4S7NFXXG> and Android <https://github.com/notifications/mobile/android/AU3A7URHMIS6UI7FWYGFGET5WHBMBA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTMNBWGYZTEMJVGAY2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2K4ZTPN52GK4S7MFXGI4TPNFSA>. Download it today! You are receiving this because you authored the thread.Message ID: ***@***.***>

famosab

Waiting for the underlying tool to expose the CLI :)

Lukecele · 2026-10-28T11:38:18Z

Hi @famosab, thank you again for the guidance. I have prepared the CLI change in a separate review PR so the package-side change can be checked independently before updating this module: Lukecele/spatial-game-assay#1

The PR adds the insitugame console entry point, preserves the current JSON/CSV output contract, and passes CI on Python 3.9–3.13. Once the package change is reviewed and released, I will generate the matching Seqera container and update this nf-core module.

Copilot

🟡 Changes recommended

The module reports completion without running the assay or exporting analytical results, and its advertised H5AD input format is incompatible with the CSV-only loader.

2 open issues

What changed in this PR

Introduces an insitugame module intended to support spatial evolutionary game analysis in nf-core pipelines, with dependency configuration, metadata and tests.

Changes:

Adds a Nextflow process with JSON/CSV outputs, version reporting and stub behaviour.
Pins insitugame to version 1.0.6 and documents the module’s interface.
Adds regular and stub tests with recorded snapshots and test tags.

File	Description
`modules/nf-core/insitugame/tests/tags.yml`	Associates the module path with its test tag.
`modules/nf-core/insitugame/tests/main.nf.test.snap`	Records regular and stub output snapshots.
`modules/nf-core/insitugame/tests/main.nf.test`	Adds TSV-based regular and stub tests.
`modules/nf-core/insitugame/meta.yml`	Documents the tool, inputs, outputs and maintainers.
`modules/nf-core/insitugame/main.nf`	Defines the process, metadata-only outputs and stub.
`modules/nf-core/insitugame/environment.yml`	Pins the Bioconda dependency to 1.0.6.

_{Review effort: Balanced}

+    'Resistant': resistant,
+    'Stroma': stroma
+}
+assay = InSituGameAssay(gene_spec)


Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot

🔵 Needs a closer look

The module reports completion without running inference, producing metadata and row counts instead of the promised payoff matrices and Nash equilibria.

1 open issue

Run assay analysis and export fitted analytical results

1 resolved since last review

Remove unsupported .h5ad input format advertisement

_{Review effort: Balanced}

Lukecele · 2026-10-28T12:51:45Z

I re-read the module and found a correctness issue beyond the container/CLI migration: the previous inline code instantiated InSituGameAssay but never called fit(), so the JSON/CSV outputs were metadata-only. I corrected this in the CLI follow-up PR: Lukecele/spatial-game-assay#2 (20 package tests pass).

I am holding the nf-core branch update until that release is published to Bioconda: the current latest release/container is v1.0.6 and cannot contain the merged CLI, so changing the module to a guessed 1.0.7 Wave image would break Conda/Docker/Singularity. Once Bioconda has 1.0.7, I will generate the matching Seqera references, replace the inline Python with insitugame, update the snapshot, and rerun all module tests. Thanks @famosab for pointing me to the CLI/Wave approach.

new module: insitugame

3d75761

github-actions Bot added the size/m label Oct 27, 2026

Luca and others added 4 commits October 27, 2026 14:01

fix(insitugame): format meta.yml with prettier to satisfy pre-commit …

ef6c2b2

…hook

ci: re-trigger tests after quay container publication

14a67bb

Merge branch 'master' into insitugame

0acad1a

chore: trigger CI to bypass GHCR network timeout

b9c0786

chore(insitugame): add license header to main.nf

74aeedb

Merge branch 'master' into insitugame

18ba733

famosab reviewed Oct 28, 2026

View reviewed changes

famosab requested changes Oct 28, 2026

View reviewed changes

Lukecele mentioned this pull request Oct 28, 2026

feat: add CLI for nf-core integration Lukecele/spatial-game-assay#1

Merged

Lukecele requested review from famosab and a balanced review from Copilot October 28, 2026 12:24

Copilot started reviewing on behalf of Lukecele October 28, 2026 12:25

View session

Copilot AI reviewed Oct 28, 2026

View reviewed changes

Comment thread modules/nf-core/insitugame/main.nf

'Resistant': resistant,

'Stroma': stroma

}

assay = InSituGameAssay(gene_spec)

Comment thread modules/nf-core/insitugame/meta.yml Outdated

Update file pattern for spatial_data in meta.yml

24de8ad

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Lukecele requested a balanced review from Copilot October 28, 2026 12:39

Copilot started reviewing on behalf of Lukecele October 28, 2026 12:39

View session

Copilot AI reviewed Oct 28, 2026

View reviewed changes

Navigation Menu

Repository navigation

new module: insitugame - #13471

new module: insitugame#13471
Lukecele wants to merge 8 commits into
nf-core:masterfrom
Lukecele:insitugame

Lukecele commented Oct 27, 2026 •

edited

Loading

Uh oh!

Lukecele commented Oct 27, 2026

Uh oh!

Lukecele commented Oct 27, 2026

Uh oh!

famosab Oct 28, 2026

Uh oh!

famosab Oct 28, 2026

Uh oh!

Lukecele commented Oct 28, 2026

Uh oh!

famosab commented Oct 28, 2026

Uh oh!

Lukecele commented Oct 28, 2026 via email

Uh oh!

famosab left a comment

Uh oh!

Lukecele commented Oct 28, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Lukecele commented Oct 28, 2026

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		'https://depot.galaxyproject.org/singularity/insitugame:1.0.6--pyhdfd78af_0':
		'quay.io/biocontainers/insitugame:1.0.6--pyhdfd78af_0' }"

Conversation

Lukecele commented Oct 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Verification & Compliance

Checklist

Uh oh!

Lukecele commented Oct 27, 2026

Uh oh!

Lukecele commented Oct 27, 2026

Uh oh!

famosab Oct 28, 2026

Choose a reason for hiding this comment

Uh oh!

famosab Oct 28, 2026

Choose a reason for hiding this comment

Uh oh!

Lukecele commented Oct 28, 2026

Uh oh!

famosab commented Oct 28, 2026

Uh oh!

Lukecele commented Oct 28, 2026 via email

Uh oh!

famosab left a comment

Choose a reason for hiding this comment

Uh oh!

Lukecele commented Oct 28, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🔵 Needs a closer look

Uh oh!

Lukecele commented Oct 28, 2026

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Lukecele commented Oct 27, 2026 •

edited

Loading