Closed ndodda-amazon closed 3 years ago
Merging #495 (ce33550) into master (1d373b9) will increase coverage by
8.69%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## master #495 +/- ##
==========================================
+ Coverage 63.92% 72.61% +8.69%
==========================================
Files 165 115 -50
Lines 13055 10375 -2680
==========================================
- Hits 8345 7534 -811
+ Misses 4710 2841 -1869
Impacted Files | Coverage Δ | |
---|---|---|
smdebug/xgboost/utils.py | 0.00% <0.00%> (-14.76%) |
:arrow_down: |
smdebug/xgboost/hook.py | 0.00% <0.00%> (-4.35%) |
:arrow_down: |
smdebug/core/access_layer/s3.py | 91.54% <0.00%> (-4.23%) |
:arrow_down: |
smdebug/core/reader.py | 85.18% <0.00%> (-3.71%) |
:arrow_down: |
smdebug/exceptions.py | 65.47% <0.00%> (-2.39%) |
:arrow_down: |
smdebug/core/access_layer/file.py | 96.00% <0.00%> (-2.00%) |
:arrow_down: |
smdebug/core/tfrecord/tensor_reader.py | 95.45% <0.00%> (-1.52%) |
:arrow_down: |
smdebug/core/locations.py | 86.11% <0.00%> (-1.39%) |
:arrow_down: |
smdebug/core/tensor.py | 79.03% <0.00%> (-1.21%) |
:arrow_down: |
smdebug/core/index_reader.py | 90.08% <0.00%> (-0.44%) |
:arrow_down: |
... and 60 more |
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 1d373b9...ce33550. Read the comment docs.
Description of changes:
Fix bug in #491 to also skip XGBoost rules tests in CI.
Style and formatting:
I have run
pre-commit install
to ensure that auto-formatting happens with every commit.Issue number, if available
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.