Skip to content

Conversation

@ch-kr
Copy link
Contributor

@ch-kr ch-kr commented Jun 3, 2020

This is necessary for VCF export and isn't added by any other functions in vep.py

@ch-kr ch-kr requested a review from jkgoodrich June 3, 2020 16:54
Copy link
Contributor

@jkgoodrich jkgoodrich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if we should just store everything after "Consequence annotations from Ensembl VEP. Format:" maybe as VEP_CSQ_FIELDS or something along those lines. Then it could also be used as default to vep_struct_to_csq. Then another VEP_CSQ_HEADER that is "Consequence annotations from Ensembl VEP. Format:" + VEP_CSQ_FIELDS.

@ch-kr ch-kr requested a review from jkgoodrich June 3, 2020 21:00
Copy link
Contributor

@jkgoodrich jkgoodrich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me.

@ch-kr ch-kr requested a review from jkgoodrich June 9, 2020 18:58
@ch-kr ch-kr merged commit d11fd8d into master Jun 9, 2020
@ch-kr ch-kr deleted the vep branch June 9, 2020 19:02
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.

3 participants