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

Removes unnecessary and buggy validation check #8580

Merged

Added missing initialization

3c9f358
Select commit
Loading
Failed to load commit list.
Merged

Removes unnecessary and buggy validation check #8580

Added missing initialization
3c9f358
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Nov 12, 2023 in 0s

86.104% (-0.244%) compared to 2900e01

View this Pull Request on Codecov

86.104% (-0.244%) compared to 2900e01

Details

Codecov Report

Merging #8580 (3c9f358) into master (2900e01) will decrease coverage by 0.244%.
The diff coverage is 80.000%.

@@               Coverage Diff               @@
##              master     #8580       +/-   ##
===============================================
- Coverage     86.349%   86.104%   -0.244%     
+ Complexity     39541     39414      -127     
===============================================
  Files           2362      2362               
  Lines         186157    186152        -5     
  Branches       20303     20302        -1     
===============================================
- Hits          160744    160285      -459     
- Misses         18261     18744      +483     
+ Partials        7152      7123       -29     
Files Coverage Δ
...tools/walkers/haplotypecaller/AlleleFiltering.java 58.848% <100.000%> (ø)
...institute/hellbender/utils/read/FlowBasedRead.java 51.288% <100.000%> (-0.305%) ⬇️
...tute/hellbender/utils/read/FlowBasedReadUtils.java 45.026% <0.000%> (ø)

... and 25 files with indirect coverage changes