Skip to content

[WIP] Balanced Random Forest#8728

Closed
massich wants to merge 17 commits intoscikit-learn:masterfrom
massich:is/8607
Closed

[WIP] Balanced Random Forest#8728
massich wants to merge 17 commits intoscikit-learn:masterfrom
massich:is/8607

Conversation

@massich
Copy link
Copy Markdown
Contributor

@massich massich commented Apr 11, 2017

Reference Issue

Fixes #8607

What does this implement/fix? Explain your changes.

This PR takes over #5181

Any other comments?

@massich massich changed the title [WIP] Is/8607 [WIP] Balanced Random Forest (#8607, #5181) Apr 11, 2017
@massich massich changed the title [WIP] Balanced Random Forest (#8607, #5181) [WIP] Balanced Random Forest Apr 11, 2017
@potash
Copy link
Copy Markdown

potash commented Apr 11, 2017

@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.

@glemaitre
Copy link
Copy Markdown
Member

What's your plan for tests?

@potash We want to extend the benchmark on several imbalanced dataset.
In this regard, we will use the datasets which we recently added in the imbalanced-learning as well as the additional metrics available.

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.

Could you reset the history to avoid merging master into your branch. Rebasing should be the way.
In fact, an easy thing could be to checkout the current branch and cherry-pick your PEP8 commits and make a PR on @massich branch.

Let me know how I can help.

I am sure that we will need support to make sense of the benchmarking results.

@potash
Copy link
Copy Markdown

potash commented Apr 11, 2017

@glemaitre I cherry-picked the flake8 fixes and put them in a PR to @masich.

massich and others added 2 commits April 12, 2017 10:50
Change `balance_data=Ture` for `class_weight='balanced_bootstraping'`. See scikit-learn#8607
discussion
@massich massich closed this Apr 12, 2017
@massich massich deleted the is/8607 branch April 12, 2017 12:51
@massich massich mentioned this pull request Apr 12, 2017
4 tasks
@massich
Copy link
Copy Markdown
Contributor Author

massich commented Apr 12, 2017

This work is followed here #8732 'cos I messed up somehow and I don't find howto retake the PR

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.

bootstrapping based on sample weights in random forests

3 participants