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

Attach VirtualOffsets to SamReads #75

Merged
merged 2 commits into from
Apr 17, 2015
Merged

Attach VirtualOffsets to SamReads #75

merged 2 commits into from
Apr 17, 2015

Conversation

danvk
Copy link
Collaborator

@danvk danvk commented Apr 17, 2015

This will allow de-duping them in BamDataSource after fetching.

This simplifies some code in BamFile.

See #26

Review on Reviewable

@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 93.25% when pulling d0f4fd8 on bam-glue into cdb2e66 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.4%) to 93.61% when pulling 2037f41 on bam-glue into cdb2e66 on master.

danvk added a commit that referenced this pull request Apr 17, 2015
Attach VirtualOffsets to SamReads
@danvk danvk merged commit 1b0ae83 into master Apr 17, 2015
@danvk danvk deleted the bam-glue branch June 22, 2015 16:19
Georgehe4 pushed a commit to Georgehe4/pileup.js that referenced this pull request Apr 28, 2017
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.

None yet

2 participants