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

Refactor com.intel.genomicsdb package references to org.genomicsdb #5587

Merged
merged 1 commit into from
Jan 17, 2019

Conversation

nalinigans
Copy link
Contributor

@nalinigans nalinigans commented Jan 17, 2019

See Issue 5277 - Migrate to org.genomicsdb fork.

The first genomicsdb 1.0.0.beta jar consists of only a refactoring of all the packages to org.genomicsdb. Note that this pass should have no performance implications compared to the last Intel release as there is no change other than refactoring.

Issues 5568-buffer resizing excessive logging and 5342-file synching error will both be addressed in the next release.

@nalinigans nalinigans self-assigned this Jan 17, 2019
@codecov-io
Copy link

codecov-io commented Jan 17, 2019

Codecov Report

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

@@              Coverage Diff               @@
##             master     #5587       +/-   ##
==============================================
+ Coverage     87.04%   87.042%   +0.002%     
- Complexity    31490     31491        +1     
==============================================
  Files          1924      1924               
  Lines        145189    145189               
  Branches      16082     16082               
==============================================
+ Hits         126373    126376        +3     
+ Misses        12958     12955        -3     
  Partials       5858      5858
Impacted Files Coverage Δ Complexity Δ
...e/hellbender/tools/genomicsdb/GenomicsDBUtils.java 86.047% <ø> (ø) 8 <0> (ø) ⬇️
...ls/genomicsdb/GenomicsDBImportIntegrationTest.java 88.47% <ø> (ø) 82 <0> (ø) ⬇️
...institute/hellbender/engine/FeatureDataSource.java 76.471% <ø> (ø) 42 <0> (ø) ⬇️
.../hellbender/tools/genomicsdb/GenomicsDBImport.java 78.195% <ø> (ø) 53 <0> (ø) ⬇️
...e/hellbender/engine/GenomicsDBIntegrationTest.java 97.959% <ø> (ø) 11 <0> (ø) ⬇️
...der/tools/genomicsdb/GenomicsDBImportUnitTest.java 86.207% <ø> (ø) 9 <0> (ø) ⬇️
...lotypecaller/readthreading/ReadThreadingGraph.java 88.608% <0%> (-0.253%) 144% <0%> (-1%)
...ithwaterman/SmithWatermanIntelAlignerUnitTest.java 60% <0%> (ø) 2% <0%> (ø) ⬇️
...nder/utils/runtime/StreamingProcessController.java 67.773% <0%> (+0.474%) 33% <0%> (ø) ⬇️
...utils/smithwaterman/SmithWatermanIntelAligner.java 80% <0%> (+30%) 3% <0%> (+2%) ⬆️

Copy link
Collaborator

@droazen droazen left a comment

Choose a reason for hiding this comment

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

👍 looks good -- merging

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.

3 participants