-
Notifications
You must be signed in to change notification settings - Fork 4
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
Compatibility with phased assembly variant caller? #237
Comments
Hi @jessmewald this type of I/O-related issues are strange because SvAnna uses HtsJDK library to read VCF file, and HtsJDK is developed by the authors/maintainers of VCF format. So, this looks like an issue with the input VCF file on the first glance. Are you 100% sure the white spaces are consistent? Sometimes a VCF file contains spaces ( Some text editors have an option for showing white space characters. Then, a VCF file can look like this:
A valid VCF should use
Can you please verify that your VCF, especially the offending line, uses the right column delimiters? |
Yes, the lines on the screenshot look OK. Can you please share the file |
Heres a link to the file on onedrive. Let me know if you can't access. |
Hi @jessmewald I think this is not related to the source of the VCF (PAV) but it's an issue of VCF compression. The file you shared worked OK when uncompressed. I made a new release with a fix. Can you please try it out and let me know if you run into any issues? Thanks a lot and all the best.. :) |
Hi @jessmewald thanks a lot for the update. I think you're right about the I think I have a fix and I pushed the code to You would need to build SvAnna from sources, as described in the docs here with a little difference. You would need to switch the branch before building (see the extra line below). So, something like this should work: git clone https://github.com/TheJacksonLaboratory/SvAnna
cd SvAnna
git checkout report-unfiltered-variants-in-html && git pull # <-- the extra line
./mvnw package After the build, you should get a distribution ZIP in the Regarding the other issues - by default, issues with weird variants or genes are logged and the variants are ignored, in order to finish the analysis. It's hard to know how serious these issues are without additional context. Please let me know if you suspect something odd is going on. |
Thanks Daniel! We are now able to annotate and prioritize the PAV VCFs with SvAnna using the updated code in the |
Hi there, thanks for developing such an easy to use tool. We would like to run SvAnna with VCF outputs from PAV. It seems that SvAnna is truncating the vcfs in a way that is not obvious. We are running SvAnna 1.0.3 with PAV 2.2.4 (also developed at JAX).
We've tried a variety of PAV VCFs, and the SvAnna error is consistent among them. We get the following error:
For every VCF generated with PAV, SvAnna will stop reading the file at a line that it reports to not have enough columns. In all cases, the line it fails to read does contain the same number of columns, in the same format, as every other line.
Any help you can offer to trouble shoot this error would be greatly appreciated! Thanks for your time.
The text was updated successfully, but these errors were encountered: