-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix short data CV #70
base: master
Are you sure you want to change the base?
Fix short data CV #70
Conversation
@pavelkrizek FYI |
Codecov Report
@@ Coverage Diff @@
## master #70 +/- ##
==========================================
- Coverage 93.79% 93.73% -0.07%
==========================================
Files 56 56
Lines 2853 2888 +35
==========================================
+ Hits 2676 2707 +31
- Misses 177 181 +4
Continue to review full report at Codecov.
|
@MichalChromcak Thanks for catching and fixing the bug! Everything is good from my side, just the function |
In certain cases, the current behaviour of the
scorer
stores thecv_data
wrongly (see the split numbers). Further plotting and evaluation functionality is partially affected by that.This MR aims to fix this.
Example: