Skip to content

DOC AdaboostRegressor do not require the base estimator to support sample weight#14407

Merged
glemaitre merged 3 commits intoscikit-learn:masterfrom
qinhanmin2014:adaboostregressor
Jul 19, 2019
Merged

DOC AdaboostRegressor do not require the base estimator to support sample weight#14407
glemaitre merged 3 commits intoscikit-learn:masterfrom
qinhanmin2014:adaboostregressor

Conversation

@qinhanmin2014
Copy link
Copy Markdown
Member

We sample the training set according to the weight, which is consistent with the reference.

Copy link
Copy Markdown
Member

@glemaitre glemaitre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Could catch

Support for sample weighting is required. If ``None``, then
the base estimator is ``DecisionTreeRegressor(max_depth=3)``
If ``None``, then the base estimator is
``DecisionTreeRegressor(max_depth=3)``.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
``DecisionTreeRegressor(max_depth=3)``.
:class:`sklearn.tree.DecisionTreeRegressor(max_depth=3)`.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will this work? let's see

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this doesn't work @glemaitre because there're some extra parameters

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh right. So let's merge.

@glemaitre glemaitre merged commit 054c6f9 into scikit-learn:master Jul 19, 2019
@qinhanmin2014 qinhanmin2014 deleted the adaboostregressor branch July 19, 2019 13:55
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.

2 participants