Thanks to visit codestin.com
Credit goes to github.com

Skip to content

DOC Ensures that sklearn.preprocessing._data.binarize passes numpydoc validation #22801

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.

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

Merged
merged 1 commit into from
Mar 12, 2022

Conversation

magalimorin18
Copy link
Contributor

@magalimorin18 magalimorin18 commented Mar 12, 2022

Reference Issues/PRs

Addresses #21350

What does this implement/fix? Explain your changes.

Any other comments?

.gitignore Outdated
@@ -10,6 +10,8 @@
build
sklearn/datasets/__config__.py
sklearn/**/*.html
.env
Copy link
Member

Choose a reason for hiding this comment

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

I'm happy with this change, but it needs its own PR

Copy link
Member

Choose a reason for hiding this comment

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

Indeed, two PRs got mixed. We fixed

@@ -1868,3 +1868,36 @@ def test_random_trees_embedding_feature_names_out():
]
]
assert_array_equal(expected_names, names)

# def test_rf_regressor_prediction_range():
Copy link
Member

Choose a reason for hiding this comment

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

I think you didn't mean to comment these tests in this PR

Copy link
Member

@GaelVaroquaux GaelVaroquaux left a comment

Choose a reason for hiding this comment

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

LGTM.

As these changes are documentation only, I will merge once tests are green.

Copy link
Member

@GaelVaroquaux GaelVaroquaux left a comment

Choose a reason for hiding this comment

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

LGTM.

As these changes are documentation only, I will merge once tests are green.

@chagaz
Copy link
Contributor

chagaz commented Mar 14, 2022

Adding the #pariswimlds hashtag :-)

glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Apr 6, 2022
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.

4 participants