-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
[WIP] Balanced Random Forest #8728
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
Conversation
@massich thanks so much for resurrecting this PR-- I'm still using the feature in multiple projects and so would love to see it incorporated. I just pushed some flake8 fixes to my original PR #5181. What's your plan for tests? I have a set of real-world benchmarks in my brf-example repo. Let me know how I can help. |
@potash We want to extend the benchmark on several imbalanced dataset.
Could you reset the history to avoid merging master into your branch. Rebasing should be the way.
I am sure that we will need support to make sense of the benchmarking results. |
@glemaitre I cherry-picked the flake8 fixes and put them in a PR to @masich. |
Feature/balanced random forest
Change `balance_data=Ture` for `class_weight='balanced_bootstraping'`. See scikit-learn#8607 discussion
This work is followed here #8732 'cos I messed up somehow and I don't find howto retake the PR |
Reference Issue
Fixes #8607
What does this implement/fix? Explain your changes.
This PR takes over #5181
Any other comments?