aboutsummaryrefslogtreecommitdiff
path: root/R/pkg
diff options
context:
space:
mode:
authorXin Ren <iamshrek@126.com>2016-09-10 09:52:53 -0700
committerShivaram Venkataraman <shivaram@cs.berkeley.edu>2016-09-10 09:52:53 -0700
commit71b7d42f5fa8d3a891ce831eedb308d1f407dd7e (patch)
treef704568ad71c18a79d275448e47b58535b14fe3f /R/pkg
parent6ea5055fa734d435b5f148cf52d3385a57926b60 (diff)
downloadspark-71b7d42f5fa8d3a891ce831eedb308d1f407dd7e.tar.gz
spark-71b7d42f5fa8d3a891ce831eedb308d1f407dd7e.tar.bz2
spark-71b7d42f5fa8d3a891ce831eedb308d1f407dd7e.zip
[SPARK-16445][MLLIB][SPARKR] Fix @return description for sparkR mlp summary() method
## What changes were proposed in this pull request? Fix summary() method's `return` description for spark.mlp ## How was this patch tested? Ran tests locally on my laptop. Author: Xin Ren <iamshrek@126.com> Closes #15015 from keypointt/SPARK-16445-2.
Diffstat (limited to 'R/pkg')
-rw-r--r--R/pkg/R/mllib.R6
-rw-r--r--R/pkg/inst/tests/testthat/test_mllib.R2
2 files changed, 5 insertions, 3 deletions
diff --git a/R/pkg/R/mllib.R b/R/pkg/R/mllib.R
index f8d1095a49..234b208166 100644
--- a/R/pkg/R/mllib.R
+++ b/R/pkg/R/mllib.R
@@ -720,8 +720,9 @@ setMethod("predict", signature(object = "MultilayerPerceptronClassificationModel
# Returns the summary of a Multilayer Perceptron Classification Model produced by \code{spark.mlp}
#' @param object a Multilayer Perceptron Classification Model fitted by \code{spark.mlp}
-#' @return \code{summary} returns a list containing \code{layers}, the label distribution, and
-#' \code{tables}, conditional probabilities given the target label.
+#' @return \code{summary} returns a list containing \code{labelCount}, \code{layers}, and
+#' \code{weights}. For \code{weights}, it is a numeric vector with length equal to
+#' the expected given the architecture (i.e., for 8-10-2 network, 100 connection weights).
#' @rdname spark.mlp
#' @export
#' @aliases summary,MultilayerPerceptronClassificationModel-method
@@ -732,7 +733,6 @@ setMethod("summary", signature(object = "MultilayerPerceptronClassificationModel
labelCount <- callJMethod(jobj, "labelCount")
layers <- unlist(callJMethod(jobj, "layers"))
weights <- callJMethod(jobj, "weights")
- weights <- matrix(weights, nrow = length(weights))
list(labelCount = labelCount, layers = layers, weights = weights)
})
diff --git a/R/pkg/inst/tests/testthat/test_mllib.R b/R/pkg/inst/tests/testthat/test_mllib.R
index ac896cfbcf..5b1404c621 100644
--- a/R/pkg/inst/tests/testthat/test_mllib.R
+++ b/R/pkg/inst/tests/testthat/test_mllib.R
@@ -369,6 +369,8 @@ test_that("spark.mlp", {
expect_equal(summary$labelCount, 3)
expect_equal(summary$layers, c(4, 5, 4, 3))
expect_equal(length(summary$weights), 64)
+ expect_equal(head(summary$weights, 5), list(-0.878743, 0.2154151, -1.16304, -0.6583214, 1.009825),
+ tolerance = 1e-6)
# Test predict method
mlpTestDF <- df