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

Insert_size calculation bug #95

Open
PfaffS opened this issue Feb 16, 2018 · 6 comments
Open

Insert_size calculation bug #95

PfaffS opened this issue Feb 16, 2018 · 6 comments
Assignees
Labels

Comments

@PfaffS
Copy link
Collaborator

PfaffS commented Feb 16, 2018

When in the scr-ref.bam no paired reads are mapped, the script scale-reads.pl will divide by 0.
It tries to print the mean of all paired mapped reads template lengths (TLEN 9th column in .bam). If there are no paired reads it will try to calculate it by dividing by 0.

@PfaffS
Copy link
Collaborator Author

PfaffS commented Feb 16, 2018

@PfaffS PfaffS self-assigned this Feb 16, 2018
@greatfireball greatfireball self-assigned this Feb 16, 2018
@PfaffS PfaffS added the bug label Feb 16, 2018
@greatfireball
Copy link
Member

I would like to add a test for that bug to guarantee its fix inside the source code.
Due to I only need to test scale-reads.pl i will need some reads which just map single end. Are your reads available? I do require about 10k read pairs.

@greatfireball
Copy link
Member

Reopened because of missing test

@greatfireball greatfireball reopened this Feb 16, 2018
@greatfireball
Copy link
Member

Are you able to provide a testset @PfaffS ?

@greatfireball
Copy link
Member

Just bumping up the issue again @PfaffS

How about writing a test?

@greatfireball
Copy link
Member

And once more @PfaffS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants