support String type INFO fields with multiple values #87

Merged
merged 1 commit into from Jan 30, 2013

Conversation

Projects
None yet
2 participants
@alimanfoo

I have String type INFO fields with multiple values, but PyVCF currently does not split these by the comma char into multiple values, it returns a single string. I see that there is a special exception for String type INFO fields in the code handling multiple values, so maybe you have a good reason for doing this (some legacy VCFs that have commas in INFO fields where they shouldn't?) But the VCF 4.1 spec does state that comma is reserved in INFO fields values for specifying multiple values, so the current behaviour is not standard (and I have to add special cases in my VCF-handling code to work around).

jamescasbon pushed a commit that referenced this pull request Jan 30, 2013

James Casbon
Merge pull request #87 from alimanfoo/master
support String type INFO fields with multiple values

@jamescasbon jamescasbon merged commit 1dbe5b5 into jamescasbon:master Jan 30, 2013

gotgenes pushed a commit to gotgenes/PyVCF that referenced this pull request May 13, 2014

James Casbon
Merge pull request #87 from alimanfoo/master
support String type INFO fields with multiple values
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment