Skip to content

Support list-valued VCF fields in datasheets - #160

Merged
maxulysse merged 2 commits into
devfrom
feat/vcf-lists
Oct 13, 2026
Merged

maxulysse merged 2 commits into
devfrom
feat/vcf-lists

Conversation

@maxulysse

Copy link
Copy Markdown
Member

Summary

Adapts the pipeline to the recent nf-core/references-datasheets updates (PRs #18, #19, #21), which introduced list-valued VCF fields and removed the multi-dbsnps test datasheets.

  • schema_references.json: vcf_*_vcf and vcf_*_vcf_tbi fields now accept a single string or an array; TBI arrays allow null entries (meaning the index needs to be built). Added the missing ascat_loci_rt field (was consumed by the channel logic but silently dropped during validation).
  • datasheet_to_channel: replaced the five copy-pasted VCF blocks with a toVcfChannel(datasheet, type) helper that normalizes string-or-list values, pairs VCFs with TBIs positionally, and sets run_tabix per VCF.
  • tests/multiple.nf.test: repointed the tabix test from the deleted GRCh37_GRCh38_multiple_dbsnps_globs.yml to igenomes/GATK/GRCh37decoy.yml, which covers list-valued VCFs with a null TBI entry (forces exactly one tabix run).

Testing

  • 7/7 nf-tests pass: sarek ×2, multiple ×2, htslib ×2, default ×1
  • Sarek snapshots unchanged (byte-identical); multiple snapshot re-recorded for the new datasheet
  • prek run --all-files passes

Generated by OpenCode

Accept string or list values for vcf_*_vcf and vcf_*_vcf_tbi schema
fields, with null entries allowed in TBI lists. Pair VCFs with indexes
positionally in datasheet_to_channel so each VCF gets its own run_tabix
flag, and add the missing ascat_loci_rt schema entry. Repoint the tabix
test to igenomes/GATK/GRCh37decoy.yml after the multi-dbsnps datasheets
were removed upstream.

Generated by OpenCode
@github-actions

github-actions Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

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

Posted for pipeline commit 496332c

+| ✅ 216 tests passed       |+
#| ❔  12 tests were ignored |#
!| ❗   8 tests had warnings |!
Details

❗ Test warnings:

  • files_exist - File not found: .github/workflows/awsfulltest.yml
  • 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 methods_description_template.yml: #Update the HTML below to your preferred methods description, e.g. add publication citation for this pipeline
  • pipeline_todos - TODO string in base.config: Check the defaults for all processes
  • pipeline_todos - TODO string in usage.md: Add documentation about anything specific to running your pipeline. For general topics, please point to (and add to) the main nf-core website.
  • pipeline_todos - TODO string in output.md: Write this documentation describing your workflow's output

❔ Tests ignored:

✅ Tests passed:

Run details

  • nf-core/tools version 4.1.0
  • Run at 2026-10-13 12:13:27

@github-actions

github-actions Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

❌ nf-test failed with latest Nextflow version

Note

Tests with Nextflow's latest version failed but it will not cause a CI workflow failure.
Please check if the failure is expected with newer (edge-)releases of Nextflow or if it needs fixing.

  • ❌ docker | latest-everything | Shard 6/7

See the full run for details.

@maxulysse
maxulysse merged commit c68aad9 into dev Oct 13, 2026
48 of 52 checks passed
@maxulysse
maxulysse deleted the feat/vcf-lists branch October 13, 2026 13:38
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