Skip to content

DOC Remove old docstring notes from sklearn/feature_extraction/image.py#28468

Merged
betatim merged 1 commit intoscikit-learn:mainfrom
ChVeen:Remove-old-docstring-notes-in-feature_extraction/image.py
Feb 20, 2024
Merged

DOC Remove old docstring notes from sklearn/feature_extraction/image.py#28468
betatim merged 1 commit intoscikit-learn:mainfrom
ChVeen:Remove-old-docstring-notes-in-feature_extraction/image.py

Conversation

@ChVeen
Copy link
Copy Markdown
Contributor

@ChVeen ChVeen commented Feb 19, 2024

Reference Issues/PRs

Fixes issue #28454

What does this implement/fix? Explain your changes.

This PR removes too old notes which are contained in two docstrings at

https://github.com/scikit-learn/scikit-learn/blob/main/sklearn/feature_extraction/image.py#L178-L185

and

https://github.com/scikit-learn/scikit-learn/blob/main/sklearn/feature_extraction/image.py#L232-L239

as discussed in issue ##28454.

Any other comments?

@github-actions
Copy link
Copy Markdown

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 4c2d63f. Link to the linter CI: here

Copy link
Copy Markdown
Contributor

@Charlie-XIAO Charlie-XIAO left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the cleanup @ChVeen!

@betatim betatim merged commit 874165b into scikit-learn:main Feb 20, 2024
@betatim
Copy link
Copy Markdown
Member

betatim commented Feb 20, 2024

Thank you for tidying up!

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