-
Notifications
You must be signed in to change notification settings - Fork 86
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
Default number of cv_folds is changed to 5 #865
Conversation
Hello @andreygetmanov! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2022-09-08 11:35:05 UTC |
Codecov Report
@@ Coverage Diff @@
## master #865 +/- ##
==========================================
+ Coverage 87.56% 87.89% +0.32%
==========================================
Files 196 196
Lines 13405 13405
==========================================
+ Hits 11738 11782 +44
+ Misses 1667 1623 -44
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Посмотри хотя бы в дебаге на всякий случай, что там теперь действительно 5 фолдов. А то мало ли
52c15e9
to
62f4727
Compare
Changed default cv_folds number in params for table data and in some tests