Skip to content

DOC: Fix typos in the scikit-learn Examples#32241

Merged
adrinjalali merged 6 commits intoscikit-learn:mainfrom
star1327p:example-typos
Sep 23, 2025
Merged

DOC: Fix typos in the scikit-learn Examples#32241
adrinjalali merged 6 commits intoscikit-learn:mainfrom
star1327p:example-typos

Conversation

@star1327p
Copy link
Copy Markdown
Contributor

Reference Issues/PRs

What does this implement/fix? Explain your changes.

Fix typos in the scikit-learn Examples

Any other comments?

@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 21, 2025

✔️ Linting Passed

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

Generated for commit: 7ee90ca. Link to the linter CI: here

# Notice that the most important hyperparameters values were tuned using a grid
# search procedure not shown in this notebook for the sake of simplicity. See
# the example script
# :ref:`sphx_glr_auto_examples_model_selection_plot_grid_search_text_feature_extraction.py` # noqa: E501
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think lint is failing due to removal of this

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@lucyleeow Yes, but the "# noqa: E501" should not appear in the web version of documentation. Is there a way to remove it without triggering an error?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Unfortunately I don't think so sphinx-gallery/sphinx-gallery#1403

I was worried that a general regex to remove such things would be brittle but I could just remove specifically the E501 as that seems to be the common problem.

Either way it's a sphinx gallery issue to solve and I think we have to keep it in scikit learn.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@lucyleeow I added back the "# noqa: E501" string. Hope the compilation works this time!

@adrinjalali adrinjalali merged commit 306a55e into scikit-learn:main Sep 23, 2025
36 checks passed
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