Skip to content

improve estimator checks - #174

Open
kulbachcedric wants to merge 1 commit into
mainfrom
162-fix-saving-and-loading
Open

kulbachcedric wants to merge 1 commit into
mainfrom
162-fix-saving-and-loading

Conversation

@kulbachcedric

Copy link
Copy Markdown
Collaborator

No description provided.

@kulbachcedric kulbachcedric linked an issue Aug 28, 2026 that may be closed by this pull request
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 98.27586% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.35%. Comparing base (9abeb0b) to head (3cbdb0c).

Files with missing lines Patch % Lines
deep_river/utils/estimator_checks.py 98.18% 2 Missing ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #174      +/-   ##
==========================================
+ Coverage   82.57%   83.35%   +0.78%     
==========================================
  Files          27       27              
  Lines        2089     2097       +8     
==========================================
+ Hits         1725     1748      +23     
+ Misses        364      349      -15     
Files with missing lines Coverage Δ
deep_river/base.py 85.23% <100.00%> (-0.71%) ⬇️
deep_river/utils/estimator_checks.py 72.29% <98.18%> (+5.26%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@codspeed

codspeed Bot commented Aug 28, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 75 untouched benchmarks


Comparing 162-fix-saving-and-loading (3cbdb0c) with main (9abeb0b)

Open in CodSpeed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix saving and loading

2 participants