-
Notifications
You must be signed in to change notification settings - Fork 28k
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-11755] [R] SparkR should export "predict" #9732
Conversation
Test build #45993 has finished for PR 9732 at commit
|
@@ -1054,6 +1054,10 @@ setGeneric("year", function(x) { standardGeneric("year") }) | |||
#' @export | |||
setGeneric("glm") | |||
|
|||
#' @rdname predict | |||
#' @export | |||
setGeneric("predict", function(object, newData) { standardGeneric("predict") }) |
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.
It may be better to keep the signature of "predict" to be same as that in R base.
setGeneric("predict", function(object, ...) { standardGeneric("predict") })
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.
Yeah we should follow the same signature
Test build #46087 has finished for PR 9732 at commit
|
LGTM |
1 similar comment
LGTM |
Merging this to master, branch-1.6 |
The bug described at [SPARK-11755](https://issues.apache.org/jira/browse/SPARK-11755), after exporting ```predict``` we can both get the help information from the SparkR and base R package like the following: ```Java > help(predict) Help on topic ‘predict’ was found in the following packages: Package Library SparkR /Users/yanboliang/data/trunk2/spark/R/lib stats /Library/Frameworks/R.framework/Versions/3.2/Resources/library Choose one 1: Make predictions from a model {SparkR} 2: Model Predictions {stats} ``` Author: Yanbo Liang <ybliang8@gmail.com> Closes #9732 from yanboliang/spark-11755. (cherry picked from commit 8fb775b) Signed-off-by: Shivaram Venkataraman <shivaram@cs.berkeley.edu>
The bug described at SPARK-11755, after exporting
predict
we can both get the help information from the SparkR and base R package like the following: