NOJIRA: minor fixes
Project: http://git-wip-us.apache.org/repos/asf/mahout/repo Commit: http://git-wip-us.apache.org/repos/asf/mahout/commit/bf1cdd42 Tree: http://git-wip-us.apache.org/repos/asf/mahout/tree/bf1cdd42 Diff: http://git-wip-us.apache.org/repos/asf/mahout/diff/bf1cdd42 Branch: refs/heads/flink-binding Commit: bf1cdd428c02552cb27c87de89c63a896d1cb0e6 Parents: cc710b0 Author: smarthi <[email protected]> Authored: Sat Nov 7 21:01:55 2015 -0500 Committer: smarthi <[email protected]> Committed: Sat Nov 7 23:39:58 2015 -0500 ---------------------------------------------------------------------- .../mahout/math/scalabindings/package.scala | 3 +-- .../mahout/math/scalabindings/MathSuite.scala | 20 +++++++++----------- .../org/apache/mahout/math/SparseMatrix.java | 1 - 3 files changed, 10 insertions(+), 14 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mahout/blob/bf1cdd42/math-scala/src/main/scala/org/apache/mahout/math/scalabindings/package.scala ---------------------------------------------------------------------- diff --git a/math-scala/src/main/scala/org/apache/mahout/math/scalabindings/package.scala b/math-scala/src/main/scala/org/apache/mahout/math/scalabindings/package.scala index 7ff09bf..426996b 100644 --- a/math-scala/src/main/scala/org/apache/mahout/math/scalabindings/package.scala +++ b/math-scala/src/main/scala/org/apache/mahout/math/scalabindings/package.scala @@ -19,7 +19,6 @@ package org.apache.mahout.math import org.apache.mahout.math.solver.EigenDecomposition import collection._ -import JavaConversions._ /** * Mahout matrices and vectors' scala syntactic sugar @@ -197,7 +196,7 @@ package object scalabindings { * @return */ def svec(sdata: TraversableOnce[(Int, AnyVal)]) = { - val cardinality = if (sdata.size > 0) sdata.map(_._1).max + 1 else 0 + val cardinality = if (sdata.nonEmpty) sdata.map(_._1).max + 1 else 0 val initialCapacity = sdata.size val sv = new RandomAccessSparseVector(cardinality, initialCapacity) sdata.foreach(t â sv.setQuick(t._1, t._2.asInstanceOf[Number].doubleValue())) http://git-wip-us.apache.org/repos/asf/mahout/blob/bf1cdd42/math-scala/src/test/scala/org/apache/mahout/math/scalabindings/MathSuite.scala ---------------------------------------------------------------------- diff --git a/math-scala/src/test/scala/org/apache/mahout/math/scalabindings/MathSuite.scala b/math-scala/src/test/scala/org/apache/mahout/math/scalabindings/MathSuite.scala index bcfe109..0503e49 100644 --- a/math-scala/src/test/scala/org/apache/mahout/math/scalabindings/MathSuite.scala +++ b/math-scala/src/test/scala/org/apache/mahout/math/scalabindings/MathSuite.scala @@ -18,14 +18,12 @@ package org.apache.mahout.math.scalabindings import org.apache.mahout.logging._ -import org.scalatest.{Matchers, FunSuite} import org.apache.mahout.math._ -import scala.math._ -import RLikeOps._ -import scala._ -import scala.util.Random +import org.apache.mahout.math.scalabindings.RLikeOps._ import org.apache.mahout.test.MahoutSuite -import org.apache.mahout.common.RandomUtils +import org.scalatest.FunSuite + +import scala.math._ class MathSuite extends FunSuite with MahoutSuite { @@ -91,9 +89,9 @@ class MathSuite extends FunSuite with MahoutSuite { val b = dense( - (0.36378319648203084), - (0.3627384439613304), - (0.2996934112658234)) + 0.36378319648203084, + 0.3627384439613304, + 0.2996934112658234) printf("B=\n%s\n", b) @@ -141,10 +139,10 @@ class MathSuite extends FunSuite with MahoutSuite { test("solve matrix-matrix") { val a = dense((1, 3), (4, 2)) - val b = dense((11), (14)) + val b = dense(11, 14) val x = solve(a, b) - val control = dense((2), (3)) + val control = dense(2, 3) (control - x).norm should be < 1e-10 } http://git-wip-us.apache.org/repos/asf/mahout/blob/bf1cdd42/math/src/main/java/org/apache/mahout/math/SparseMatrix.java ---------------------------------------------------------------------- diff --git a/math/src/main/java/org/apache/mahout/math/SparseMatrix.java b/math/src/main/java/org/apache/mahout/math/SparseMatrix.java index bf4f1a0..520778c 100644 --- a/math/src/main/java/org/apache/mahout/math/SparseMatrix.java +++ b/math/src/main/java/org/apache/mahout/math/SparseMatrix.java @@ -19,7 +19,6 @@ package org.apache.mahout.math; import com.google.common.collect.AbstractIterator; import org.apache.mahout.math.flavor.MatrixFlavor; -import org.apache.mahout.math.flavor.TraversingStructureEnum; import org.apache.mahout.math.function.DoubleDoubleFunction; import org.apache.mahout.math.function.Functions; import org.apache.mahout.math.function.IntObjectProcedure;
