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

misleading graphics output when running a concatenated reference file #274

Open
figueroakl opened this issue Dec 12, 2017 · 11 comments
Open

Comments

@figueroakl
Copy link

sent the directory I tested this on to gitter

MapvsUnmap reads does not seem to map the mapped reads percentage for some samples when running against a concatenated reference file? Could it be a sequential issue, such as, the reads got to the first reference in the file and that leads to the other references to be ignored?

@averagehat
Copy link
Contributor

mapunmapreads

@averagehat
Copy link
Contributor

pipelinetimes

samplecoverage

@averagehat
Copy link
Contributor

@averagehat
Copy link
Contributor

@averagehat
Copy link
Contributor

Looks like a bug in samtools.MPileupColumn(line)

used here

mcol = samtools.MPileupColumn(line)
<-- this is where qualdepth.json gets messed up
and here https://github.com/VDBWRAIR/ngs_mapper/blob/master/ngs_mapper/base_caller.py#L384 <-- where consensus/vcf get messed up

@averagehat
Copy link
Contributor

pileup = bqd.parse_pileup( pileup )

@averagehat
Copy link
Contributor

samtools mpileup is only returning 67 rows, all with depth=1

@averagehat
Copy link
Contributor

The problem seems to be that almost all mappings are "anomalous read pairs" or "orphans"
I'm not clear why that's happening with these samples, and with multi-entry reference fastas.
This isn't a bug, as far as I can tell.
see #112 #8

@figueroakl
Copy link
Author

I'm going to look over the issues you and try to make sense of it.

@averagehat
Copy link
Contributor

Determined the reads are orphans; a little more info is in gitter. The solution is to make allowing orphans (-A in samtools) a configurable option

@averagehat
Copy link
Contributor

Have to rework some functions, update where they are called, and pass the config around more...

base_caller: samtools.mpileup
graphsample: samtools.nogap_mpileup
bam_to_qualdepth: samtools.nogap_mpileup; bam.get_refstats; bam.alignment_info;
tagreads: samtools.view (x2)-- can just add A=True
stats_at_refpos: samtools.mpileup
bqd is clear of system calls

tagreads - gets config, sort of
base_caller gets config
graphsample does not get config (?)

flagstats?

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