Closed peterdudfield closed 1 year ago
Merging #40 (accebf1) into main (0071a25) will increase coverage by
11.55%
. The diff coverage is93.75%
.:exclamation: Current head accebf1 differs from pull request most recent head b5bcb39. Consider uploading reports for the commit b5bcb39 to get more accurate results
@@ Coverage Diff @@
## main #40 +/- ##
===========================================
+ Coverage 33.77% 45.32% +11.55%
===========================================
Files 20 21 +1
Lines 915 920 +5
===========================================
+ Hits 309 417 +108
+ Misses 606 503 -103
Impacted Files | Coverage Δ | |
---|---|---|
gradboost_pv/app.py | 0.00% <0.00%> (ø) |
|
gradboost_pv/__init__.py | 100.00% <100.00%> (ø) |
|
gradboost_pv/inference/models.py | 91.46% <100.00%> (+44.22%) |
:arrow_up: |
gradboost_pv/save.py | 100.00% <100.00%> (ø) |
... and 3 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Pull Request
Description
add mock tests for saving and running model
Fixes #
How Has This Been Tested?
CI tests
Checklist: