-
-
Notifications
You must be signed in to change notification settings - Fork 25.9k
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
Conversation
.gitignore
Outdated
@@ -10,6 +10,8 @@ | |||
build | |||
sklearn/datasets/__config__.py | |||
sklearn/**/*.html | |||
.env |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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(): |
There was a problem hiding this comment.
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
0d60bf2
to
f7ebeee
Compare
There was a problem hiding this 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.
There was a problem hiding this 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.
Adding the #pariswimlds hashtag :-) |
Co-authored-by: Magali Morin <[email protected]>
Reference Issues/PRs
Addresses #21350
What does this implement/fix? Explain your changes.
Any other comments?