Skip to content

feat: add VCF extension inference support#207

Open
VedantMadane wants to merge 1 commit intowheretrue:mainfrom
VedantMadane:add-vcf-extension-inference
Open

feat: add VCF extension inference support#207
VedantMadane wants to merge 1 commit intowheretrue:mainfrom
VedantMadane:add-vcf-extension-inference

Conversation

@VedantMadane
Copy link
Copy Markdown

Summary

Fixes #154

Adds support for inferring VCF file type from file extension, completing the extension inference feature for all major bioinformatics formats.

Changes

  • Added file_extension parameter to VCFReadOptions::try_new()
  • Updated From impl to pass extension to ListingVCFTableOptions
  • Added test_vcf_extension_inference test to verify automatic VCF detection

Format Support Status

  • fasta (previously done)
  • fastq (previously done)
  • gff (previously done)
  • bed (previously done)
  • vcf (this PR)

Testing

  • New test case for VCF extension inference
  • Existing tests pass
  • Verified with .vcf and .vcf.gz files

- Allow file_extension parameter in VCFReadOptions constructor
- Pass file_extension to ListingVCFTableOptions in From impl
- Add test for VCF extension inference from file path

This enables reading VCF files without explicitly specifying the
file extension or compression type - they are inferred from the path.

Contributes to wheretrue#154
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.

Infer extension and compression from file path.

1 participant