Skip to content

Commit a8120d2

Browse files
committed
[SPARK-4409] Finished updates to API according to SPARK-4614
1 parent f798c82 commit a8120d2

File tree

2 files changed

+21
-13
lines changed

2 files changed

+21
-13
lines changed

mllib/src/main/scala/org/apache/spark/mllib/linalg/Matrices.scala

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,6 @@ package org.apache.spark.mllib.linalg
1919

2020
import breeze.linalg.{Matrix => BM, DenseMatrix => BDM, CSCMatrix => BSM}
2121

22-
import org.apache.spark.util.random.XORShiftRandom
23-
import org.apache.spark.util.Utils
24-
2522
import java.util.{Random, Arrays}
2623
import scala.collection.mutable.ArrayBuffer
2724

mllib/src/test/java/org/apache/spark/mllib/linalg/JavaMatricesSuite.java

Lines changed: 21 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -21,25 +21,34 @@
2121
import org.junit.Test;
2222

2323
import java.io.Serializable;
24+
import java.util.Random;
2425

2526
public class JavaMatricesSuite implements Serializable {
2627

2728
@Test
2829
public void randMatrixConstruction() {
29-
Matrix r = Matrices.rand(3, 4, 24);
30-
DenseMatrix dr = DenseMatrix.rand(3, 4, 24);
30+
Random rng = new Random(24);
31+
Matrix r = Matrices.rand(3, 4, rng);
32+
rng.setSeed(24);
33+
DenseMatrix dr = DenseMatrix.rand(3, 4, rng);
3134
assertArrayEquals(r.toArray(), dr.toArray(), 0.0);
3235

33-
Matrix rn = Matrices.randn(3, 4, 24);
34-
DenseMatrix drn = DenseMatrix.randn(3, 4, 24);
36+
rng.setSeed(24);
37+
Matrix rn = Matrices.randn(3, 4, rng);
38+
rng.setSeed(24);
39+
DenseMatrix drn = DenseMatrix.randn(3, 4, rng);
3540
assertArrayEquals(rn.toArray(), drn.toArray(), 0.0);
3641

37-
Matrix s = Matrices.sprand(3, 4, 0.5, 24);
38-
SparseMatrix sr = SparseMatrix.sprand(3, 4, 0.5, 24);
42+
rng.setSeed(24);
43+
Matrix s = Matrices.sprand(3, 4, 0.5, rng);
44+
rng.setSeed(24);
45+
SparseMatrix sr = SparseMatrix.sprand(3, 4, 0.5, rng);
3946
assertArrayEquals(s.toArray(), sr.toArray(), 0.0);
4047

41-
Matrix sn = Matrices.sprandn(3, 4, 0.5, 24);
42-
SparseMatrix srn = SparseMatrix.sprandn(3, 4, 0.5, 24);
48+
rng.setSeed(24);
49+
Matrix sn = Matrices.sprandn(3, 4, 0.5, rng);
50+
rng.setSeed(24);
51+
SparseMatrix srn = SparseMatrix.sprandn(3, 4, 0.5, rng);
4352
assertArrayEquals(sn.toArray(), srn.toArray(), 0.0);
4453
}
4554

@@ -95,8 +104,10 @@ public void concatenateMatrices() {
95104
int m = 3;
96105
int n = 2;
97106

98-
SparseMatrix spMat1 = SparseMatrix.sprand(m, n, 0.5, 42);
99-
DenseMatrix deMat1 = DenseMatrix.rand(m, n, 42);
107+
Random rng = new Random(42);
108+
SparseMatrix spMat1 = SparseMatrix.sprand(m, n, 0.5, rng);
109+
rng.setSeed(42);
110+
DenseMatrix deMat1 = DenseMatrix.rand(m, n, rng);
100111
Matrix deMat2 = Matrices.eye(3);
101112
Matrix spMat2 = Matrices.speye(3);
102113
Matrix deMat3 = Matrices.eye(2);

0 commit comments

Comments
 (0)