Thanks to visit codestin.com
Credit goes to github.com

Skip to content

DOC: Fixes broken link to scikit-learn-intelex documentation in install.rst page - #34640

Merged
ogrisel merged 3 commits into
scikit-learn:mainfrom
ShubhanC:fix/sklearn-intelex-link
Aug 3, 2026
Merged

DOC: Fixes broken link to scikit-learn-intelex documentation in install.rst page#34640
ogrisel merged 3 commits into
scikit-learn:mainfrom
ShubhanC:fix/sklearn-intelex-link

Conversation

@ShubhanC

@ShubhanC ShubhanC commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Fixing a Broken Link!!

Reference Issues/PRs

Does not fix an given issue, since the fix is an obvious change. From the website:

Unless you’re only fixing typos or obvious inconsistencies in the documentation, make sure the PR relates to at least one existing issue that is ready for contribution.

What does this implement/fix? Explain your changes.

Near the bottom of the (installation page)[https://scikit-learn.org/stable/install.html] is a section titled Intel Extension for Scikit-learn. The excerpt to focus on is:

Note that those solvers are not enabled by default, please refer to the scikit-learn-intelex documentation for more details on usage scenarios. Direct export example:

The hyperlink throws an Error 404. Now, they have a new website at https://uxlfoundation.github.io/scikit-learn-intelex/latest/patching.html, which is what this PR replaces the old link with.

First time contributor introduction

This is my first contribution! Hi everyone, my name is Shubhan, and I'm studying Statistics and Computer Science at the University of Illinois. I've used scikit-learn for over two years in my university and professional career, and I can't wait to give back to the community!

AI usage disclosure

I used AI assistance for:

  • Research and understanding

Any other comments?

There will likely be more PRs (with issues) for either documentation changes or updates. I firmly believe that documentation should be clear and informative, so there will be some more involved documentation changes soon!

old link lead to an error 404 message, replacing the old link with a working documentation link.
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

Thank you for opening your first pull request to scikit-learn! 🎉

To help get your contribution reviewed, please make sure that:

  • You have filled out the pull request template.
  • The pull request addresses an existing issue that is ready for
    contribution (e.g. not tagged as "Needs Triage", "Needs Decision", ...).
    If you are proposing a new feature, please open an issue to discuss it first.
  • There are no other open pull requests already targeting the same issue.
  • You have followed the pull request checklist.
    In particular, linting and tests should pass.

@ogrisel

ogrisel commented Aug 3, 2026

Copy link
Copy Markdown
Member

Thanks for the fix. Merging.

@ogrisel
ogrisel merged commit 8fa0e81 into scikit-learn:main Aug 3, 2026
38 checks passed
prady0t pushed a commit to prady0t/scikit-learn that referenced this pull request Sep 2, 2026
@jeremiedbb jeremiedbb mentioned this pull request Sep 8, 2026
14 tasks
jeremiedbb pushed a commit to jeremiedbb/scikit-learn that referenced this pull request Sep 8, 2026
jeremiedbb pushed a commit to jeremiedbb/scikit-learn that referenced this pull request Sep 9, 2026
jeremiedbb pushed a commit to jeremiedbb/scikit-learn that referenced this pull request Sep 9, 2026
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.

2 participants