samtools / htsjdk

A Java API for high-throughput sequencing data (HTS) formats.
http://samtools.github.io/htsjdk/
283 stars 242 forks source link

Yf add interval list codec #1327

Closed yfarjoun closed 5 years ago

yfarjoun commented 5 years ago
codecov-io commented 5 years ago

Codecov Report

Merging #1327 into master will increase coverage by 0.307%. The diff coverage is 75%.

@@               Coverage Diff               @@
##              master     #1327       +/-   ##
===============================================
+ Coverage     67.822%   68.129%   +0.307%     
- Complexity      8253      8406      +153     
===============================================
  Files            562       563        +1     
  Lines          33644     34260      +616     
  Branches        5640      5778      +138     
===============================================
+ Hits           22818     23341      +523     
- Misses          8654      8720       +66     
- Partials        2172      2199       +27
Impacted Files Coverage Δ Complexity Δ
.../java/htsjdk/samtools/util/BufferedLineReader.java 68.421% <ø> (ø) 6 <0> (ø) :arrow_down:
...c/main/java/htsjdk/samtools/util/IntervalList.java 73.897% <100%> (-0.613%) 69 <0> (-6)
src/main/java/htsjdk/samtools/util/Interval.java 61.29% <33.333%> (-2.995%) 28 <1> (+1)
...main/java/htsjdk/variant/vcf/AbstractVCFCodec.java 73.846% <60%> (+0.081%) 86 <1> (ø) :arrow_down:
...htsjdk/tribble/IntervalList/IntervalListCodec.java 79.31% <79.31%> (ø) 14 <14> (?)
src/main/java/htsjdk/samtools/cram/CRAIIndex.java 86.813% <0%> (-1.593%) 39% <0%> (+13%)
...a/htsjdk/samtools/cram/build/ContainerFactory.java 95.122% <0%> (-1.545%) 6% <0%> (+1%)
...jdk/samtools/cram/build/CramContainerIterator.java 88.889% <0%> (-1.111%) 16% <0%> (+6%)
...java/htsjdk/samtools/cram/structure/Container.java 90.411% <0%> (-0.065%) 42% <0%> (+20%)
...mtools/cram/build/CramContainerHeaderIterator.java 100% <0%> (ø) 4% <0%> (+2%) :arrow_up:
... and 13 more
yfarjoun commented 5 years ago

put in a indexed test

yfarjoun commented 5 years ago

back to you @cmnbroad

yfarjoun commented 5 years ago

@cmnbroad better?