Skip to content

Implement symphony integration and reference mapping#291

Open
nictru wants to merge 18 commits into
devfrom
symphony
Open

Implement symphony integration and reference mapping#291
nictru wants to merge 18 commits into
devfrom
symphony

Conversation

@nictru
Copy link
Copy Markdown
Collaborator

@nictru nictru commented May 29, 2026

No description provided.

nictru and others added 9 commits May 28, 2026 15:59
Introduces SYMPHONY_INTEGRATE module and wires it into INTEGRATE as
integration_methods symphony, with nf-test coverage and docs.
Requires symphonypy 0.2.3 for harmonypy 0.2 compatibility (symphonypy#9).
Update symphonypy to 0.2.4 with a rebuilt Wave image, and use importlib.metadata for package versions since symphonypy does not expose __version__.

Co-authored-by: Cursor <cursoragent@cursor.com>
Align the process name with symphonypy's harmony_integrate entry point and update references and nf-test snapshots.

Co-authored-by: Cursor <cursoragent@cursor.com>
Route the harmony integration method through SYMPHONY_HARMONYINTEGRATE, remove the duplicate symphony option, and delete the old scanpy/harmony module.

Co-authored-by: Cursor <cursoragent@cursor.com>
The extension_base artifacts now live in nf-core/test-datasets, so the temporary build and collection scripts are no longer needed in this repo.
Users now select symphony in integration_methods and pass symphony_reference for query mapping, with outputs published under combine/integrate/symphony.
Use the upstream extension_base commit for reference mapping, extension, and sub-atlas tests instead of branch URLs or the temporary fork.
Regenerated module, subworkflow, and pipeline snapshots with real H5AD outputs from the new official test datasets.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 29, 2026

nf-core pipelines lint overall result: Passed ✅ ⚠️

Posted for pipeline commit 8631fd4

+| ✅ 289 tests passed       |+
#| ❔   1 tests had warnings |#
!| ❗  15 tests had warnings |!
Details

❗ Test warnings:

  • files_exist - File not found: conf/igenomes.config
  • files_exist - File not found: conf/igenomes_ignored.config
  • readme - README contains the placeholder zenodo.XXXXXXX. This should be replaced with the zenodo doi (after the first release).
  • pipeline_todos - TODO string in README.md: Add citation for pipeline after first release. Uncomment lines below and update Zenodo doi and badge at the top of this file.
  • pipeline_todos - TODO string in README.md: Add bibliography of tools and data used in your pipeline
  • pipeline_todos - TODO string in nextflow.config: Optionally, you can add a pipeline-specific nf-core config at https://github.com/nf-core/configs
  • pipeline_todos - TODO string in main.nf: Optionally add in-text citation tools to this list.
  • pipeline_todos - TODO string in main.nf: Optionally add bibliographic entries to this list.
  • pipeline_todos - TODO string in main.nf: Only uncomment below if logic in toolCitationText/toolBibliographyText has been filled!
  • pipeline_todos - TODO string in awsfulltest.yml: You can customise AWS full pipeline tests as required
  • pipeline_todos - TODO string in CONTRIBUTING.md: Add any pipeline specific contribution guidelines here, such as coding styles, procedures, checklists etc.
  • pipeline_todos - TODO string in nextflow.config: Specify any additional parameters here
  • pipeline_todos - TODO string in base.config: Check the defaults for all processes
  • pipeline_todos - TODO string in base.config: Customise requirements for specific processes.
  • pipeline_todos - TODO string in methods_description_template.yml: #Update the HTML below to your preferred methods description, e.g. add publication citation for this pipeline

❔ Tests fixed:

✅ Tests passed:

Run details

  • nf-core/tools version 4.0.2
  • Run at 2026-06-01 21:02:32

nictru added 9 commits May 29, 2026 23:19
Point pipeline nf-tests back at the forked fixtures needed by existing tests and align the COMBINE test inputs with the current workflow signature.
Support gene_ids and gene_id in filtered matrices and fall back to var index when neither is present, fixing AMBIENT_CORRECTION tests on symbol-only h5ad inputs.
Add bioconductor-glmgampoi and bump r-seurat so SCTransform uses the supported v2 backend in the Wave image.
Run SCANPY_FILTER on raw counts before HVG selection and refresh integration and combat snapshots.
Surface actionable context when mygene.info returns server or network errors instead of a raw httpx traceback.
Pass species from the pipeline into unify and quality control so gene ID conversion respects the configured organism, and update tests and parameter docs accordingly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant