Skip to content

Test and improve Nextflow pipeline: fix bugs, add stubs, add CI - #15

Draft
bruno-ariano with Copilot wants to merge 6 commits into
nextflow_branchfrom
copilot/refactor-unique-script-modules
Draft

Test and improve Nextflow pipeline: fix bugs, add stubs, add CI#15
bruno-ariano with Copilot wants to merge 6 commits into
nextflow_branchfrom
copilot/refactor-unique-script-modules

Conversation

Copilot AI commented Mar 5, 2026

Copy link
Copy Markdown

The Nextflow pipeline had broken CLI flags in modules/regions/main.nf, EXPORT_REGIONS was imported but never wired into the workflow, and there was no way to validate the pipeline DAG without containers or real data.

Bug fixes

  • modules/regions/main.nf: removed nonexistent --workers flag; --table_regions--table-regions; --tiledb_lb_typesumstat--type_sumstat; output pattern was emitting locusbreaker filenames instead of region CSV output
  • main.nf: added missing if (params.regions) block inside if (params.export) so EXPORT_REGIONS is actually invoked

Stub blocks

Added stub: blocks to all 8 modules (create_tiledb, ingestion, merge_metadata, snp, regions, locusbreaker, traits, recompute_meta), enabling DAG validation without containers:

nextflow run main.nf -profile test_ingest         -stub
nextflow run main.nf -profile test_export_snp     -stub
nextflow run main.nf -profile test_export_regions -stub
nextflow run main.nf -profile test_export_traits  -stub
nextflow run main.nf -profile test_export_lb      -stub
nextflow run main.nf -profile test_recompute_meta -stub

Test config (conf/test.config)

  • test_recompute_meta: replaced hardcoded cluster paths with ${projectDir}/example_data/recompute_meta_test_table.csv
  • test_export_traits: points to new trait_list_test.csv (correct TRAIT column format)
  • test_export_regions: params aligned with fixed module

New example data files

File Purpose
example_data/example_data_table_test.csv Documents ingestion column format without cluster-specific paths
example_data/trait_list_test.csv TRAIT column (Tgd:ENSG…) for traits export tests
example_data/recompute_meta_test_table.csv CELL,CHR format for recompute-metadata tests

CI (.github/workflows/ci.yml)

Three jobs on every push/PR: Python pytest, Nextflow stub tests for all 6 profiles, and a full integration test that runs ingestion with -process.container null.

Docs

  • docs/README.md: full rewrite — parameter tables for every workflow mode, module reference table, stub/integration testing guide
  • README.md: added Nextflow testing section

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…with tests

Co-authored-by: bruno-ariano <26384813+bruno-ariano@users.noreply.github.com>
Copilot AI changed the title [WIP] Refactor functions into modules for better organization Refactor: split monolithic export command into focused modules + add pytest suite Mar 5, 2026
… 32 tests + README update

Co-authored-by: bruno-ariano <26384813+bruno-ariano@users.noreply.github.com>
Copilot AI changed the title Refactor: split monolithic export command into focused modules + add pytest suite Refactor: split monolithic harmonize_ingest.py and export.py into focused sub-modules Mar 5, 2026
… docs

Co-authored-by: bruno-ariano <26384813+bruno-ariano@users.noreply.github.com>
Copilot AI changed the title Refactor: split monolithic harmonize_ingest.py and export.py into focused sub-modules Refactor ingestion into sub-modules, add Nextflow tests and CI, fix pipeline bugs Mar 5, 2026
Copilot AI changed the title Refactor ingestion into sub-modules, add Nextflow tests and CI, fix pipeline bugs Test and improve Nextflow pipeline: fix bugs, add stubs, add CI Mar 7, 2026
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.

2 participants