Closed peterdudfield closed 1 year ago
Merging #33 (a718f9e) into main (5529e9e) will increase coverage by
1.78%
. The diff coverage is77.14%
.:exclamation: Current head a718f9e differs from pull request most recent head 6bdccfd. Consider uploading reports for the commit 6bdccfd to get more accurate results
:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@ Coverage Diff @@
## main #33 +/- ##
==========================================
+ Coverage 32.10% 33.88% +1.78%
==========================================
Files 19 20 +1
Lines 869 900 +31
==========================================
+ Hits 279 305 +26
- Misses 590 595 +5
Impacted Files | Coverage Δ | |
---|---|---|
gradboost_pv/app.py | 0.00% <0.00%> (ø) |
|
gradboost_pv/inference/data_feeds.py | 40.40% <0.00%> (ø) |
|
gradboost_pv/models/s3.py | 34.21% <0.00%> (ø) |
|
gradboost_pv/models/utils.py | 91.83% <0.00%> (ø) |
|
gradboost_pv/utils/logger.py | 60.00% <ø> (ø) |
|
gradboost_pv/inference/utils.py | 96.29% <96.29%> (ø) |
|
gradboost_pv/__init__.py | 100.00% <100.00%> (ø) |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Pull Request
Description
add small logging
Fixes #
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist: