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

Added a new WEX test bam to src/test/resources/large, with a companion target interval list #4756

Merged
merged 1 commit into from May 10, 2018

Conversation

droazen
Copy link
Collaborator

@droazen droazen commented May 9, 2018

This is a ~130 MB snippet of the NexPond-377866 NA12878 bam from
https://github.com/broadinstitute/palantir/wiki/Gold-Standard-Datasets
with:

-all reads from chromosomes 20 and 21
-50,000 unmapped reads added to the end

It can be used with the existing human_g1k_v37.20.21.fasta reference in
src/test/resources/large

Tests that use this new file will be coming in a future branch.

…n target interval list

This is a ~130 MB snippet of the NexPond-377866 NA12878 bam from
https://github.com/broadinstitute/palantir/wiki/Gold-Standard-Datasets
with:

-all reads from chromosomes 20 and 21
-50,000 unmapped reads added to the end

It can be used with the existing human_g1k_v37.20.21.fasta reference in
src/test/resources/large

Tests that use this new file will be coming in a future branch.
Copy link
Member

@lbergelson lbergelson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@lbergelson lbergelson assigned droazen and unassigned lbergelson May 9, 2018
@codecov-io
Copy link

Codecov Report

Merging #4756 into master will increase coverage by 0.011%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##              master     #4756       +/-   ##
===============================================
+ Coverage     79.987%   79.998%   +0.011%     
  Complexity     17421     17421               
===============================================
  Files           1081      1081               
  Lines          63119     63119               
  Branches       10196     10196               
===============================================
+ Hits           50487     50494        +7     
+ Misses          8648      8641        -7     
  Partials        3984      3984
Impacted Files Coverage Δ Complexity Δ
...nder/utils/runtime/StreamingProcessController.java 72.016% <0%> (+2.469%) 50% <0%> (ø) ⬇️
...utils/smithwaterman/SmithWatermanIntelAligner.java 90% <0%> (+10%) 3% <0%> (ø) ⬇️

@lbergelson
Copy link
Member

@droazen one thing I just noticed, the header of this includes the full hg19 dict, do we want to replace it with just 20/21?

@droazen droazen merged commit 42c0db3 into master May 10, 2018
@droazen droazen deleted the dr_add_exome_test_data branch May 10, 2018 15:48
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

3 participants