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

Mapq #55

Open
nchernia opened this issue May 1, 2018 · 5 comments
Open

Mapq #55

nchernia opened this issue May 1, 2018 · 5 comments

Comments

@nchernia
Copy link
Contributor

nchernia commented May 1, 2018

I’ve noticed that running bwa meth results in a higher rate of MAPQ 0 reads than vanilla bwa (I did a test on non-bisulfite converted data). Is there any way to ameliorate this or is it just a natural consequence of needing to distinguish between methylated and nonmethylated loci?

@brentp
Copy link
Owner

brentp commented May 1, 2018

this is the only place it's adjust mapq and it's setting to 1. mapq 0 means it maps to multiple places equally which is more likely to occur with a simplified reference.

@nchernia
Copy link
Contributor Author

nchernia commented May 1, 2018

Yes, I was wondering on a more fundamental level; perhaps via the flags bwa-meth uses to call bwa. I asked Heng Li and he suggested asking here.

BTW, what's the motivation behind that heuristic? Those are reads that bwa deems good enough to map, is there a reason more fundamental to bisulfite alignment that you would label them failing QC?

@brentp
Copy link
Owner

brentp commented May 1, 2018

if you can show that it performs better without that, I'll remove it. I've actually found (for non-BS-Seq data) that alignments/ regions with high NM are often bad as well.

@nchernia
Copy link
Contributor Author

nchernia commented May 1, 2018

I was thinking it should be an optional flag to turn it off (was eventually going to code it and make a pull request). We expect chimeric reads in our experiment, but if one was not expecting chimeras, I could see it being useful.

@brentp
Copy link
Owner

brentp commented May 1, 2018

still same criteria. if you can show that it is more accurate without, it can be removed. I'm hesitant to add more command-line flags.

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

2 participants