Skip to content

DOC adding suggested spaces in multioutput/RegressorChain#17498

Merged
thomasjpfan merged 3 commits intoscikit-learn:masterfrom
marenwestermann:regressor-chain
Jun 6, 2020
Merged

DOC adding suggested spaces in multioutput/RegressorChain#17498
thomasjpfan merged 3 commits intoscikit-learn:masterfrom
marenwestermann:regressor-chain

Conversation

@marenwestermann
Copy link
Copy Markdown
Member

Reference Issues/PRs

References #15215
Towards #3846

What does this implement/fix? Explain your changes.

Adds suggested spaces by reviewer

Any other comments?

cc: (pair programming partner) @LeandroLuque

Copy link
Copy Markdown
Member

@amueller amueller left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Copy Markdown
Member

@thomasjpfan thomasjpfan left a comment

Choose a reason for hiding this comment

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

LGTM

@thomasjpfan
Copy link
Copy Markdown
Member

Thank you @marenwestermann and @LeandroLuque!

@thomasjpfan thomasjpfan merged commit 5a85627 into scikit-learn:master Jun 6, 2020
@reshamas
Copy link
Copy Markdown
Member

reshamas commented Jun 6, 2020

#DataUmbrella sprint

@marenwestermann marenwestermann deleted the regressor-chain branch June 12, 2020 12:55
viclafargue pushed a commit to viclafargue/scikit-learn that referenced this pull request Jun 26, 2020
…rn#17498)

* example for multioutput regressor

* add spaces (suggested changes) #DataUmbrella

Co-authored-by: Tim Nonner <tim.nonner@tamedia.ch>
jayzed82 pushed a commit to jayzed82/scikit-learn that referenced this pull request Oct 22, 2020
…rn#17498)

* example for multioutput regressor

* add spaces (suggested changes) #DataUmbrella

Co-authored-by: Tim Nonner <tim.nonner@tamedia.ch>
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.

5 participants