Skip to content

Conversation

star1327p
Copy link
Contributor

Reference Issues/PRs

What does this implement/fix? Explain your changes.

Update two more reference links.

Any other comments?

Copy link

github-actions bot commented Jul 15, 2025

✔️ Linting Passed

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

Generated for commit: 724d444. Link to the linter CI: here

Copy link
Member

@jeremiedbb jeremiedbb left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @star1327p. The CI failure is unrelated, synchronizing with main will fix it.

@@ -922,7 +922,7 @@ based on permutation of the features.
Annals of Statistics, 29, 1189-1232.

.. [Friedman2002] Friedman, J.H. (2002). `Stochastic gradient boosting.
<https://statweb.stanford.edu/~jhf/ftp/stobst.pdf>`_.
<https://doi.org/10.1016/S0167-9473(01)00065-2>`_.
Copy link
Member

Choose a reason for hiding this comment

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

This link leads to a page with a restricted access on the paper. I found this link https://citeseerx.ist.psu.edu/document?repid=rep1&type=pdf&doi=48caac2f65bce47f6d27400ae4f60d8395cec2f3, seems unrestricted.

Kalyan1210 added a commit to Kalyan1210/scikit-learn that referenced this pull request Jul 19, 2025
@Kalyan1210 Kalyan1210 mentioned this pull request Jul 19, 2025
Kalyan1210 added a commit to Kalyan1210/scikit-learn that referenced this pull request Jul 19, 2025
Kalyan1210 added a commit to Kalyan1210/scikit-learn that referenced this pull request Jul 19, 2025
Kalyan1210 added a commit to Kalyan1210/scikit-learn that referenced this pull request Jul 19, 2025
Kalyan1210 added a commit to Kalyan1210/scikit-learn that referenced this pull request Jul 19, 2025
Kalyan1210 added a commit to Kalyan1210/scikit-learn that referenced this pull request Jul 19, 2025
Kalyan1210 added a commit to Kalyan1210/scikit-learn that referenced this pull request Jul 19, 2025
Kalyan1210 added a commit to Kalyan1210/scikit-learn that referenced this pull request Jul 19, 2025
@lesteve lesteve changed the title DOC: Update two more reference links DOC Update two more reference links Jul 21, 2025
@lesteve
Copy link
Member

lesteve commented Jul 21, 2025

Merging, thanks a lot!

@lesteve lesteve merged commit 420deba into scikit-learn:main Jul 21, 2025
37 checks passed
lucyleeow pushed a commit to lucyleeow/scikit-learn that referenced this pull request Aug 22, 2025
jeremiedbb pushed a commit to jeremiedbb/scikit-learn that referenced this pull request Sep 3, 2025
@jeremiedbb jeremiedbb mentioned this pull request Sep 3, 2025
13 tasks
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