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
Fixed MarkDuplicatesSpark so it handles supplementaryReadsWithUnmappedMates properly #4785
Conversation
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.
typos, but one was there already and I don't want to wait for a repush due to a test file name which we are totally inconsistent with anyway. Looks good 👍
out.add(new Tuple2<>(pass.key(), pass)); | ||
return out.iterator(); | ||
|
||
// Mark duplicates cant properly handle templates with more than two reads in a pair |
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.
typo, missing '
in cant
File output = createTempFile("supplementaryReadUnmappedMate", "bam"); | ||
final ArgumentsBuilder args = new ArgumentsBuilder(); | ||
args.addOutput(output); | ||
args.addInput(getTestFile("supplementaryReadUnmappedmate.bam")); |
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.
weird case in the file name, but \shrug
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.
🤷♂️
Codecov Report
@@ Coverage Diff @@
## master #4785 +/- ##
==============================================
+ Coverage 80.113% 80.19% +0.078%
- Complexity 17426 17564 +138
==============================================
Files 1082 1082
Lines 63131 63525 +394
Branches 10178 10272 +94
==============================================
+ Hits 50576 50941 +365
- Misses 8567 8582 +15
- Partials 3988 4002 +14
|
Before it was resulting in a key collision.