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

output sample names in Mutect2 pon header #5733

Merged
merged 1 commit into from
Mar 1, 2019
Merged

Conversation

davidbenjamin
Copy link
Contributor

Closes #5609.

@LeeTL1220 Could you review this?

@codecov-io
Copy link

codecov-io commented Feb 27, 2019

Codecov Report

Merging #5733 into master will increase coverage by 0.003%.
The diff coverage is 100%.

@@               Coverage Diff               @@
##              master     #5733       +/-   ##
===============================================
+ Coverage     87.061%   87.064%   +0.003%     
- Complexity     31880     31884        +4     
===============================================
  Files           1940      1940               
  Lines         146804    146806        +2     
  Branches       16234     16234               
===============================================
+ Hits          127809    127815        +6     
+ Misses         13073     13070        -3     
+ Partials        5922      5921        -1
Impacted Files Coverage Δ Complexity Δ
...ls/walkers/mutect/CreateSomaticPanelOfNormals.java 94.03% <100%> (+0.184%) 22 <2> (+1) ⬆️
...ithwaterman/SmithWatermanIntelAlignerUnitTest.java 60% <0%> (ø) 2% <0%> (ø) ⬇️
...nder/utils/runtime/StreamingProcessController.java 67.773% <0%> (+0.474%) 33% <0%> (ø) ⬇️
...utils/smithwaterman/SmithWatermanIntelAligner.java 80% <0%> (+30%) 3% <0%> (+2%) ⬆️

Copy link
Contributor

@LeeTL1220 LeeTL1220 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pretty straightforward.

@davidbenjamin davidbenjamin merged commit c58707a into master Mar 1, 2019
@davidbenjamin davidbenjamin deleted the db_m2_pon_header branch March 1, 2019 18:33
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

Successfully merging this pull request may close these issues.

3 participants