-
Notifications
You must be signed in to change notification settings - Fork 28.3k
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
[SPARK-3108][MLLIB] add predictOnValues to StreamingLR and fix predictOn #2023
Conversation
QA tests have started for PR 2023 at commit
|
QA tests have finished for PR 2023 at commit
|
Jenkins, test this please. |
QA tests have started for PR 2023 at commit
|
QA tests have finished for PR 2023 at commit
|
QA tests have started for PR 2023 at commit
|
QA tests have finished for PR 2023 at commit
|
@mengxr looks good! |
It is useful in streaming to allow users to carry extra data with the prediction, for monitoring the prediction error for example. freeman-lab Author: Xiangrui Meng <[email protected]> Closes #2023 from mengxr/predict-on-values and squashes the following commits: cac47b8 [Xiangrui Meng] add classtag 2821b3b [Xiangrui Meng] use mapValues 0925efa [Xiangrui Meng] add predictOnValues to StreamingLR and fix predictOn (cherry picked from commit 217b5e9) Signed-off-by: Xiangrui Meng <[email protected]>
@freeman-lab Thanks for reviewing the code! I've merged this into master and branch-1.1. |
It is useful in streaming to allow users to carry extra data with the prediction, for monitoring the prediction error for example. freeman-lab Author: Xiangrui Meng <[email protected]> Closes apache#2023 from mengxr/predict-on-values and squashes the following commits: cac47b8 [Xiangrui Meng] add classtag 2821b3b [Xiangrui Meng] use mapValues 0925efa [Xiangrui Meng] add predictOnValues to StreamingLR and fix predictOn
It is useful in streaming to allow users to carry extra data with the prediction, for monitoring the prediction error for example. @freeman-lab