Repository navigation
Support list-valued VCF fields in datasheets - #160
Merged
Merged
Conversation
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
Generated by OpenCode
|
❌ nf-test failed with latest Nextflow versionNote Tests with Nextflow's latest version failed but it will not cause a CI workflow failure.
See the full run for details. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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.
vcf_*_vcfandvcf_*_vcf_tbifields now accept a single string or an array; TBI arrays allownullentries (meaning the index needs to be built). Added the missingascat_loci_rtfield (was consumed by the channel logic but silently dropped during validation).toVcfChannel(datasheet, type)helper that normalizes string-or-list values, pairs VCFs with TBIs positionally, and setsrun_tabixper VCF.GRCh37_GRCh38_multiple_dbsnps_globs.ymltoigenomes/GATK/GRCh37decoy.yml, which covers list-valued VCFs with anullTBI entry (forces exactly one tabix run).Testing
prek run --all-filespassesGenerated by OpenCode