Merge pull request #17199 from JuliaLang/sb/cholesky-downdate

Fixes 2 issues with the Cholesky update/downdate tests
JuliaLang · Jul 1, 2016 · 8ecd657 · 8ecd657
2 parents b2b680b + 6332269
commit 8ecd657
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/test/linalg/cholesky.jl b/test/linalg/cholesky.jl
@@ -175,9 +175,12 @@ end
 let A = complex(randn(10,5), randn(10, 5)), v = complex(randn(5), randn(5))
     for uplo in (:U, :L)
         AcA = A'A
+        BcB = AcA + v*v'
+        BcB = (BcB + BcB')/2
         F = cholfact(AcA, uplo)
-        @test LinAlg.lowrankupdate(F, v)[uplo] ≈ cholfact(AcA + v*v')[uplo]
-        @test LinAlg.lowrankdowndate(F, v)[uplo] ≈ cholfact(AcA - v*v')[uplo]
+        G = cholfact(BcB, uplo)
+        @test LinAlg.lowrankupdate(F, v)[uplo] ≈ G[uplo]
+        @test LinAlg.lowrankdowndate(G, v)[uplo] ≈ F[uplo]
     end
 end