Skip to content

DOC fix typo in load_boston docstring #21197

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 2 commits into from
Sep 30, 2021
Merged

Conversation

rdboyes
Copy link
Contributor

@rdboyes rdboyes commented Sep 29, 2021

Reference Issues/PRs

What does this implement/fix? Explain your changes.

The deprecation notice for the boston housing dataset had a sentence that read "in this case special case ..." - changed to "in this special case ..."

Any other comments?

The deprecation notice for the boston housing dataset had a sentence that read "in this case special case ..." - changed to "in this special case ..."
Copy link
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 fix @rdboyes !

May you also adjust the same wording here:

In this case special case, you can fetch the dataset from the original

@thomasjpfan thomasjpfan added this to the 1.0.1 milestone Sep 29, 2021
@thomasjpfan
Copy link
Member

Placing on 1.0.1 milestone to fix grammar in deprecated function.

@rdboyes
Copy link
Contributor Author

rdboyes commented Sep 29, 2021

Thanks @thomasjpfan - I had missed the second instance of the warning. It should be updated now to read the same way.

@glemaitre glemaitre changed the title Fix Typo in datasets/_base.py DOC fix typo in load_boston docstring Sep 30, 2021
@glemaitre glemaitre merged commit cbe4151 into scikit-learn:main Sep 30, 2021
@glemaitre
Copy link
Member

Thanks @rdboyes Merging.

@rdboyes rdboyes deleted the patch-1 branch September 30, 2021 20:42
@glemaitre glemaitre mentioned this pull request Oct 23, 2021
10 tasks
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Oct 23, 2021
samronsin pushed a commit to samronsin/scikit-learn that referenced this pull request Nov 30, 2021
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