You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mahout.apache.org by ap...@apache.org on 2016/03/15 21:36:17 UTC

[2/3] mahout git commit: MAHOUT-1807: Distributed second norm doesn't take sqrt closes apache/mahout#189

MAHOUT-1807: Distributed second norm doesn't take sqrt closes apache/mahout#189


Project: http://git-wip-us.apache.org/repos/asf/mahout/repo
Commit: http://git-wip-us.apache.org/repos/asf/mahout/commit/1e6c547e
Tree: http://git-wip-us.apache.org/repos/asf/mahout/tree/1e6c547e
Diff: http://git-wip-us.apache.org/repos/asf/mahout/diff/1e6c547e

Branch: refs/heads/flink-binding
Commit: 1e6c547e0814393413ed5c67ba149294e1229d49
Parents: f2aef44
Author: Andrew Palumbo <ap...@apache.org>
Authored: Tue Mar 15 12:44:10 2016 -0400
Committer: Andrew Palumbo <ap...@apache.org>
Committed: Tue Mar 15 16:34:50 2016 -0400

----------------------------------------------------------------------
 .../main/scala/org/apache/mahout/sparkbindings/SparkEngine.scala | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mahout/blob/1e6c547e/spark/src/main/scala/org/apache/mahout/sparkbindings/SparkEngine.scala
----------------------------------------------------------------------
diff --git a/spark/src/main/scala/org/apache/mahout/sparkbindings/SparkEngine.scala b/spark/src/main/scala/org/apache/mahout/sparkbindings/SparkEngine.scala
index b89235d..47d14db 100644
--- a/spark/src/main/scala/org/apache/mahout/sparkbindings/SparkEngine.scala
+++ b/spark/src/main/scala/org/apache/mahout/sparkbindings/SparkEngine.scala
@@ -86,12 +86,12 @@ object SparkEngine extends DistributedEngine {
     if (drm.nrow == 0) drm.colSums() else drm.colSums() /= drm.nrow
 
   override def norm[K](drm: CheckpointedDrm[K]): Double =
-    drm.rdd
+    math.sqrt(drm.rdd
       // Compute sum of squares of each vector
       .map {
       case (key, v) ⇒ v dot v
     }
-      .reduce(_ + _)
+      .reduce(_ + _))
 
 
   /** Optional engine-specific all reduce tensor operation. */