Skip to content

Conversation

@kanissh
Copy link
Contributor

@kanissh kanissh commented Jun 18, 2022

Update reference link to "A survey of Partial Least Squares (PLS) methods, with emphasis on the two-block case" by JA Wegelin

Reference Issues/PRs

#23631

What does this implement/fix? Explain your changes.

The link to the reference document "A survey of Partial Least Squares (PLS) methods, with emphasis on the two-block case" by JA Wegelin https://www.stat.washington.edu/research/reports/2000/tr371.pdf was changed to https://stat.uw.edu/sites/default/files/files/reports/2000/tr371.pdf

Any other comments?

The old link, when clicked redirects to the document in the new link location. So the old link is replaced with the new one.

Update reference link to "A survey of Partial Least Squares (PLS) methods, with emphasis on the two-block case" by JA Wegelin
@kanissh kanissh changed the title Doc fix link to reference document in Cross decomposition DOC fix link to reference document in Cross decomposition Jun 18, 2022
@lesteve
Copy link
Member

lesteve commented Jun 20, 2022

Merging, thanks a lot!

@lesteve lesteve merged commit e4d0c6d into scikit-learn:main Jun 20, 2022
@kanissh kanissh deleted the doc-tr371-#23631 branch June 20, 2022 12:52
ogrisel pushed a commit to ogrisel/scikit-learn that referenced this pull request Jul 11, 2022
Update reference link to "A survey of Partial Least Squares (PLS) methods, with emphasis on the two-block case" by JA Wegelin
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Aug 4, 2022
Update reference link to "A survey of Partial Least Squares (PLS) methods, with emphasis on the two-block case" by JA Wegelin
glemaitre pushed a commit that referenced this pull request Aug 5, 2022
Update reference link to "A survey of Partial Least Squares (PLS) methods, with emphasis on the two-block case" by JA Wegelin
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