Skip to content

DOC Ensures that make_column_transformer passes numpydoc#22183

Merged
thomasjpfan merged 9 commits intoscikit-learn:mainfrom
atharvapatil123:atharva-feature
Jan 12, 2022
Merged

DOC Ensures that make_column_transformer passes numpydoc#22183
thomasjpfan merged 9 commits intoscikit-learn:mainfrom
atharvapatil123:atharva-feature

Conversation

@atharvapatil123
Copy link
Copy Markdown
Contributor

Reference Issues/PRs

Addresses #21350

What does this implement/fix? Explain your changes.

DOC Ensures that make_column_transformer passes numpydoc validation

Any other comments?

No

Thank You

@atharvapatil123
Copy link
Copy Markdown
Contributor Author

Hi @thomasjpfan, should I do any changes here or you'll review it first?

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.

Thank you for the PR @atharvapatil123 !

We try to keep changes related to the title. In this case, please revert changes that are not in make_column_transformer and keep the changes in make_column_transformer.

>>> ct.fit_transform(X)
array([[0. , 1. , 0.5, 0.5],
[0.5, 0.5, 0. , 1. ]])
[0.5, 0.5, 0. , 1. ]]).
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.

This change is unrelated to the title of the PR. We should revert.

@atharvapatil123
Copy link
Copy Markdown
Contributor Author

Hi @thomasjpfan, I've made the necessary changes. Actually, sklearn.metrics.pairwise.manhattan_distances, and sklearn.metrics.pairwise.nan_euclidean_distances existed in the test_docstrings.py file. So, I had to remove them and commit again.

I request you to merge this pull request as soon as possible after reviewing since I need to commit my remaining two PR's and this one is conflicting in those.

Copy link
Copy Markdown
Member

@jjerphan jjerphan left a comment

Choose a reason for hiding this comment

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

LGTM.

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 thomasjpfan changed the title DOC Ensures that make_column_transformer passes numpydoc validation DOC Ensures that make_column_transformer passes numpydoc Jan 12, 2022
@thomasjpfan thomasjpfan merged commit 7f61186 into scikit-learn:main Jan 12, 2022
@atharvapatil123 atharvapatil123 deleted the atharva-feature branch January 12, 2022 17:46
@atharvapatil123 atharvapatil123 restored the atharva-feature branch January 12, 2022 18:22
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.

3 participants