-
-
Notifications
You must be signed in to change notification settings - Fork 232
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
Add PLS family algorithms #95
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did a first round of review on the PR 👍
|
||
#[cfg(test)] | ||
mod test { | ||
use super::*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the tests are currently only checking that the fitting process runs without error
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think transform
and predict
were at least covered but I guess I can add some tests checking that we get the same results as scikit-learn.
Nothing to add on my side... Well, after this first round! 😅 |
}; | ||
} | ||
|
||
// Prediction values were checked against scikit-learn 0.24.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks again for all the work! I will add a small example to the website as well
This PR implements PLS methods in a new workspace member
linfa-pls
.This is a straightforward port of scikit-learn 0.24 cross decomposition PLS code.