Skip to content
This repository has been archived by the owner on Aug 26, 2023. It is now read-only.

fix linear index access of BAM, fundamentally #395

Merged
merged 1 commit into from
Feb 17, 2017

Conversation

bicycle1885
Copy link
Member

The previous commit (049ed2d) fixes a special
case but this will fix the problem fundamentally.

The previous commit (049ed2d) fixes a special
case but this will fix the problem fundamentally.
@codecov-io
Copy link

Codecov Report

Merging #395 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #395      +/-   ##
==========================================
+ Coverage    83.7%   83.71%   +0.01%     
==========================================
  Files          97       97              
  Lines        7498     7499       +1     
==========================================
+ Hits         6276     6278       +2     
+ Misses       1222     1221       -1
Impacted Files Coverage Δ
src/intervals/index/bgzfindex.jl 96.92% <100%> (+0.04%)
src/align/alignment.jl 87.12% <ø> (+0.99%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4fcda96...ba956e4. Read the comment docs.

@bicycle1885 bicycle1885 merged commit a781f21 into BioJulia:master Feb 17, 2017
@bicycle1885 bicycle1885 deleted the fix-linindex branch February 17, 2017 04:49
bicycle1885 added a commit to bicycle1885/Bio.jl that referenced this pull request Feb 17, 2017
The previous commit (049ed2d) fixes a special
case but this will fix the problem fundamentally.
bicycle1885 added a commit to bicycle1885/Bio.jl that referenced this pull request Feb 17, 2017
The previous commit (049ed2d) fixes a special
case but this will fix the problem fundamentally.
bicycle1885 added a commit that referenced this pull request Feb 17, 2017
The previous commit (049ed2d) fixes a special
case but this will fix the problem fundamentally.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants