Closed AdamSpannbauer closed 4 years ago
Code Climate has analyzed commit 95af59c4 and detected 0 issues on this pull request.
View more on Code Climate.
Merging #91 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #91 +/- ##
======================================
Coverage 92.5% 92.5%
======================================
Files 16 16
Lines 587 587
======================================
Hits 543 543
Misses 44 44
Impacted Files | Coverage Δ | |
---|---|---|
vidstab/download_videos.py | 90% <ø> (ø) |
:arrow_up: |
vidstab/layer_utils.py | 100% <ø> (ø) |
:arrow_up: |
vidstab/cv2_utils.py | 92.85% <ø> (ø) |
:arrow_up: |
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 620d840...95af59c. Read the comment docs.
In scope changes
atol
param ofnp.allclose()
to satisfy tests.np
is the top suspect of the issue. If new updates cause tests to fall out of thisatol
then new transforms will need to be generated for test case and a note/updated min version aboutnp
.Out of scope changes
tests/pickled_transforms
dir, where data is downloaded to perform test)# nosinpection
comments for approved violations