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

Add optional summary table output to ValidateBasicSomaticShortMutations #4982

Merged
merged 2 commits into from
Jul 4, 2018

Conversation

davidbenjamin
Copy link
Contributor

With this addition, summarizing the results of the MC3 validation will use more GATK, less scripting.

@davidbenjamin
Copy link
Contributor Author

@LeeTL1220 do you mind reviewing?

@codecov-io
Copy link

Codecov Report

Merging #4982 into master will increase coverage by 0.003%.
The diff coverage is 88.889%.

@@               Coverage Diff               @@
##              master     #4982       +/-   ##
===============================================
+ Coverage     80.784%   80.787%   +0.003%     
- Complexity     17960     17966        +6     
===============================================
  Files           1095      1095               
  Lines          64592     64619       +27     
  Branches       10392     10400        +8     
===============================================
+ Hits           52180     52204       +24     
- Misses          8388      8389        +1     
- Partials        4024      4026        +2
Impacted Files Coverage Δ Complexity Δ
...tmutpileup/ValidateBasicSomaticShortMutations.java 86.905% <88.889%> (+0.94%) 13 <0> (+6) ⬆️

@davidbenjamin davidbenjamin merged commit 74bf0ec into master Jul 4, 2018
@davidbenjamin davidbenjamin deleted the db_validator_summary branch July 4, 2018 18:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants