-
Notifications
You must be signed in to change notification settings - Fork 593
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
quick fix for broken read (mapped to no ref bases) #3662
Conversation
Is there a way to drop such reads, considering it is in such an incoherent state? |
Not easy, because then we'll have a read that claims to be part of a pair but which has no mate, for example. It'll mess up other stuff. |
For documentation purposes, can you post the SAM record of the offending read here? |
Codecov Report
@@ Coverage Diff @@
## master #3662 +/- ##
===============================================
- Coverage 79.264% 79.263% -0.002%
Complexity 17394 17394
===============================================
Files 1144 1144
Lines 62930 62930
Branches 9551 9551
===============================================
- Hits 49881 49880 -1
Misses 9241 9241
- Partials 3808 3809 +1
|
EDITED_FOR_EYEBALLING |
4d88b0e
to
f39ab50
Compare
f39ab50
to
c0f77e5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:+
there is a read with the cigar 44I107S, which is complete gibberish, in the NA12878 bam.
it's nonsense, but we can't afford to blow up.