Skip to content

DOC Ensures that ColumnTransformer passes numpydoc validation #20836

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 8 commits into from
Closed

DOC Ensures that ColumnTransformer passes numpydoc validation #20836

wants to merge 8 commits into from

Conversation

baam25simo
Copy link
Contributor

@baam25simo baam25simo commented Aug 25, 2021 •
edited
Loading

Reference Issues/PRs

Addresses #20308

What does this implement/fix? Explain your changes.

  • "ColumnTransformer" removed from DOCSTRING_IGNORE_LIST in test_docstring.py
  • In "ColumnTransformer.__init__" : "See Also" section moved in the right place
  • In "ColumnTransformer.fit" : "self" description modified.
  • In "ColumnTransformer.fit_transform" : "X_t" description modified.
  • In "ColumnTransformer.transform" : "X_t" description modified.
  • In "ColumnTransformer.named_transformers_" : blank line removed
  • In "ColumnTransformer.set_params" : "**kwargs" and "self" descriptions added

@glemaitre
Copy link
Member

I merged an older similar PR that was addressing this estimator.

@glemaitre glemaitre closed this Aug 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Documentation module:compose
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@baam25simo @glemaitre