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

Make mpileup coverage parameter configurable #6

Open
messersc opened this issue Jul 5, 2018 · 0 comments
Open

Make mpileup coverage parameter configurable #6

messersc opened this issue Jul 5, 2018 · 0 comments

Comments

@messersc
Copy link

messersc commented Jul 5, 2018

Hi,

we are using EBFilter successfully, but we were recently running into trouble with some data that has very high coverage in some regions, i.e. low complexity regions like described in https://www.ncbi.nlm.nih.gov/pmc/articles/PMC4271055/

Variants in the regions should be caught by EBfilter easily I guess, but we are running into trouble with the running times of the filtering. One bottleneck that we notice is the mpileup on the normals allows for a huge depth:

https://github.com/Genomon-Project/EBFilter/blob/devel/lib/ebfilter/process_vcf.py#L64

Would it be possible to make this user configurable? Even setting it to e.g. 10000, I can't imagine there would be a huge penalty in sensitivity.

We'd be grateful for any advice,
Best regards,
Clemens

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

1 participant