Use Double.PositiveInfinity instead of Double.Max

dgshep · Aug 7, 2014 · eb1dc20 · eb1dc20
1 parent f56a882
commit eb1dc20
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/mllib/src/main/scala/org/apache/spark/mllib/linalg/distributed/RowMatrix.scala b/mllib/src/main/scala/org/apache/spark/mllib/linalg/distributed/RowMatrix.scala
@@ -413,7 +413,7 @@ class RowMatrix(
    */
   def similarColumns():
   CoordinateMatrix = {
-    similarColumnsDIMSUM(Double.MaxValue)
+    similarColumns(Double.PositiveInfinity)
   }
 
   /**
@@ -425,7 +425,7 @@ class RowMatrix(
    *              and n is the number of columns
    * @return An n x n sparse matrix of cosine similarities between columns of this matrix.
    */
-  def similarColumnsDIMSUM(gamma: Double):
+  def similarColumns(gamma: Double):
   CoordinateMatrix = {
     similarColumnsDIMSUM(columnMagnitudes(), gamma)
   }

diff --git a/mllib/src/test/scala/org/apache/spark/mllib/linalg/distributed/RowMatrixSuite.scala b/mllib/src/test/scala/org/apache/spark/mllib/linalg/distributed/RowMatrixSuite.scala
@@ -107,7 +107,7 @@ class RowMatrixSuite extends FunSuite with LocalSparkContext {
     }
 
     for (mat <- Seq(denseMat, sparseMat)) {
-      val G = mat.similarColumnsDIMSUM(150.0)
+      val G = mat.similarColumns(150.0)
       assert(closeToZero(G.toBreeze() - expected))
     }