Skip to content

Conversation

felixglush
Copy link
Contributor

Reference Issues/PRs

Addresses #20308.

What does this implement/fix? Explain your changes.

  • Added return value description to MultitaskLasso.fit(self, X, y)
  • Added descriptions to MultitaskLasso's "See Also" section (Lasso and MultiTaskElasticNet)
  • Reordered sections to match required order: Parameters, Attributes, See Also, Notes, Examples
  • Added a period (.) to parameter "selection" description and to the "See Also: MultiTaskLasso" reference

Any other comments?

@glemaitre
Copy link
Member

Thanks @felixglush LGTM.

@glemaitre glemaitre mentioned this pull request Oct 23, 2021
10 tasks
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Oct 23, 2021
samronsin pushed a commit to samronsin/scikit-learn that referenced this pull request Nov 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants