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

No FORMAT/PS description in header in SV output VCF #27

Closed
tgong1 opened this issue Apr 19, 2023 · 3 comments
Closed

No FORMAT/PS description in header in SV output VCF #27

tgong1 opened this issue Apr 19, 2023 · 3 comments

Comments

@tgong1
Copy link

tgong1 commented Apr 19, 2023

Hi,

I found there is no line for PS tag in header of SV output VCF, while SNP output VCF has the line as below:
##FORMAT=<ID=PS,Number=1,Type=Integer,Description="Phasing set (typically the position of the first variant in the set)">

This cause many problems in my downstream steps. Do you have any idea why it happens? I have tried both version 1.2 and 1.4.
Any suggestion or idea will be appreciated!

Thank you for your time and help,
Tingting

@twolinin
Copy link
Owner

twolinin commented Apr 19, 2023

Hi @tgong1

Can you help me check if there are any results in the SV output VCF using the following commands? I suspect the issue might be related to avoiding duplicate header definitions (https://github.com/twolinin/longphase/blob/v1.4/ParsingBam.cpp#L734)

grep "#" your_SV.vcf | grep "ID=PS"

Thanks

@tgong1
Copy link
Author

tgong1 commented Apr 19, 2023

Hi,

Thank you for the quick reply.
Here is the result by running the command, and I believe this is the tag from the SV merging tool SURVIVOR, which we have used.
##FORMAT=<ID=PSV,Number=1,Type=String,Description="Previous support vector">

I can remove this tag in the SV input VCF, if it can help.....

Thank you,
Tingting

@twolinin
Copy link
Owner

twolinin commented Apr 19, 2023

Hi @tgong1

This problem will be addressed in the next version update. At the moment, you can manually add the following PS definition to the header.

##FORMAT=<ID=PS,Number=1,Type=Integer,Description="Phase set identifier">

Thanks

twolinin added a commit that referenced this issue May 24, 2023
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

No branches or pull requests

3 participants