Closed rlizzo closed 5 years ago
Merging #124 into master will decrease coverage by
0.14%
. The diff coverage is87.06%
.
@@ Coverage Diff @@
## master #124 +/- ##
==========================================
- Coverage 92.18% 92.03% -0.14%
==========================================
Files 60 60
Lines 9973 10005 +32
Branches 993 997 +4
==========================================
+ Hits 9193 9208 +15
- Misses 568 582 +14
- Partials 212 215 +3
Impacted Files | Coverage Δ | |
---|---|---|
tests/test_cli.py | 100% <100%> (ø) |
:arrow_up: |
src/hangar/backends/selection.py | 100% <100%> (ø) |
:arrow_up: |
src/hangar/arrayset.py | 90.05% <100%> (+0.07%) |
:arrow_up: |
tests/test_arrayset.py | 100% <100%> (ø) |
:arrow_up: |
tests/test_checkout.py | 99.81% <100%> (ø) |
:arrow_up: |
src/hangar/utils.py | 82.99% <100%> (ø) |
:arrow_up: |
tests/test_backend_heuristics.py | 100% <100%> (ø) |
:arrow_up: |
tests/conftest.py | 97.56% <100%> (+0.01%) |
:arrow_up: |
tests/test_cli_io.py | 100% <100%> (ø) |
:arrow_up: |
src/hangar/records/commiting.py | 89.92% <33.33%> (-2.2%) |
:arrow_down: |
... and 2 more |
Other utils improvements along with reducing checkout time for large commits.
Motivation and Context
Why is this change required? What problem does it solve?:
simplify testing config and improvements born from new tiledb backend development.
Types of changes
What types of changes does your code introduce? Put an
x
in all the boxes that apply:Is this PR ready for review, or a work in progress?
How Has This Been Tested?
Put an
x
in the boxes that apply:Checklist: