-
Notifications
You must be signed in to change notification settings - Fork 0
Updates following review comments #20
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
Updates following review comments #20
Conversation
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## mobt_877_implement_qrf #20 +/- ##
=========================================================
Coverage ? 95.59%
=========================================================
Files ? 142
Lines ? 14179
Branches ? 0
=========================================================
Hits ? 13554
Misses ? 625
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| If None, then each tree contains the same number of samples as the total | ||
| available. The trees will therefore only differ due to the use of | ||
| bootstrapping (i.e. sampling with replacement) when creating each tree. | ||
| max_features (int | float): |
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.
Technically it can be a string as well, but we aren't using that.
Addresses metoppv#2135
Description
This PR:
Testing: