Skip to content

DOC Replace the phrase "where n_samples in the number" with "where n_samples is the number" #20914

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

Conversation

genvalen
Copy link
Contributor

@genvalen genvalen commented Sep 1, 2021

Reference Issues/PRs

Related to #20822

What does this implement/fix? Explain your changes.

Previously, in PR #20822, I edited a small typo in the phrase "where n_samples in the number" by updating it to "where n_samples is the number".

This PR:

  1. Edits new instances of the same typo caught by grep-searching for a version of the phrase where the parameter, n_samples, is enclosed in backticks: git grep 'where `n_samples` in' .
  2. Adds backticks to cases of n_samples (also n_features, n_components) in a docstring that don't have them:
    n_samples -> `n_samples` .

Any other comments?

I believe this catches all last cases of the typo. Please let me know if anything can be added or changed.

Copy link
Member

@TomDLT TomDLT left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

Github complains about some merge conflict, can you merge main again ?

@genvalen
Copy link
Contributor Author

genvalen commented Sep 2, 2021

Looks good, thanks!

Github complains about some merge conflict, can you merge main again ?

Yup!

@glemaitre glemaitre merged commit 6421278 into scikit-learn:main Sep 2, 2021
@glemaitre
Copy link
Member

Thanks @genvalen merging

ivannz added a commit to ivannz/scikit-learn that referenced this pull request Nov 10, 2021
ivannz added a commit to ivannz/scikit-learn that referenced this pull request Nov 10, 2021
…_new

add backticks (scikit-learn#20914), deprecate **params in fit (scikit-learn#20843), add feature_names_in_ (scikit-learn#20787)
uncompromisingly reformat plot_oneclass_vs_svdd with black
samronsin pushed a commit to samronsin/scikit-learn that referenced this pull request Nov 30, 2021
ivannz added a commit to ivannz/scikit-learn that referenced this pull request May 15, 2022
…_new

add backticks (scikit-learn#20914), deprecate **params in fit (scikit-learn#20843), add feature_names_in_ (scikit-learn#20787)
uncompromisingly reformat plot_oneclass_vs_svdd with black
ivannz added a commit to ivannz/scikit-learn that referenced this pull request Jun 14, 2022
…_new

add backticks (scikit-learn#20914), deprecate **params in fit (scikit-learn#20843), add feature_names_in_ (scikit-learn#20787)
uncompromisingly reformat plot_oneclass_vs_svdd with black
ivannz added a commit to ivannz/scikit-learn that referenced this pull request Aug 29, 2022
…_new

add backticks (scikit-learn#20914), deprecate **params in fit (scikit-learn#20843), add feature_names_in_ (scikit-learn#20787)
uncompromisingly reformat plot_oneclass_vs_svdd with black
ivannz added a commit to ivannz/scikit-learn that referenced this pull request Sep 5, 2022
…_new

add backticks (scikit-learn#20914), deprecate **params in fit (scikit-learn#20843), add feature_names_in_ (scikit-learn#20787)
uncompromisingly reformat plot_oneclass_vs_svdd with black
@genvalen genvalen deleted the edit_typos branch March 10, 2023 20:18
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