Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Id list #189

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Id list #189

wants to merge 2 commits into from

Conversation

myourshaw
Copy link

from the VCF spec: "ID - identifier: Semi-colon separated list of unique identifiers where available. If this is a dbSNP variant it is encouraged to use the rs number(s). No identifier should be present in more than one data record. If there is no identifier available, then the missing value should be used. (String, no white-space or semi-colons permitted)"

ID - identifier: Semi-colon separated list of unique identifiers where available. If this is a dbSNP variant it is encouraged to use the rs number(s). No identifier should be present in more than one data record. If there is no identifier available, then the missing value should be used. (String, no white-space or semi-colons permitted)
@martijnvermaat
Copy link
Collaborator

Thanks for the patch @myourshaw! I agree, but this will break quite some existing code.

Perhaps we should adding some tags to this and similar proposals, to make it easier to include them all in a future backwards compatibility breaking version...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants