-
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
More divide by zero errors #8
Comments
Huh. I tested it on a whole genome file without errors. I will do some more testing to get to the bottom of this. |
Could it be reference genome specific? I think the manuscript mentioned it being tested on hg19, but these samples are on hg38/GRCh38, so there's a whole different set of contigs there... |
That could be it. I'll do some testing with an hg38 aligned genome |
I guess I was wrong, since I tried running through a handful of hg19/b37 samples, and they all give me that same error. |
Thanks, I was away from my computer yesterday, will get back to you about this after some more testing today. |
I tested the code on 3 datasets and was unable to replicate your specific error. However, I added some error handling that will hopefully prevent this error from occurring. Let me if the error persists. If so, could you send me a few lines from one of your BAM files so I can see if there may be some differences in the file format? Thank you! |
I've tried this on both b37 and GRCh38 files, and a decent amount (about a dozen or so) files in total, and it happens on all of them. I'm not sure that I can send any of the data since it's all patient samples. I'll try to troubleshoot the code on my end and keep you posted, since it'd be nice to get this working. |
I ended up with another (different) set of divide by zero errors:
The interesting thing is that if I extract reads for just one chromosome, this doesn't happen. But if I run it on our full WES or WGS files, it breaks.
The text was updated successfully, but these errors were encountered: