-
-
Notifications
You must be signed in to change notification settings - Fork 26.9k
Change default n_estimators in RandomForest (to 100?) #11128
Copy link
Copy link
Closed
Labels
BlockerEasyWell-defined and straightforward way to resolveWell-defined and straightforward way to resolvegood first issueEasy with clear instructions to resolveEasy with clear instructions to resolve
Milestone
Description
Analysis of code on github shows that people use default parameters when they shouldn't. We can make that a little bit less bad by providing reasonable defaults. The default for n_estimators is not great imho and I think we should change it. I suggest 100.
We could probably run benchmarks with openml if we want to do something empirical, but I think anything is better than 10.
I'm not sure if I want to tag this 1.0 because really no-one should ever run a random forest with 10 trees imho and therefore deprecation of the current default will show people they have a bug.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
BlockerEasyWell-defined and straightforward way to resolveWell-defined and straightforward way to resolvegood first issueEasy with clear instructions to resolveEasy with clear instructions to resolve