From 4471cac8df85d622887054aa00a53bbb47d0161c Mon Sep 17 00:00:00 2001
From: Ryan Curtin <ryan@ratml.org>
Date: Tue, 25 Nov 2014 16:22:37 +0000
Subject: [PATCH] Check frobenius norm overall instead of just for one element.

---
 src/mlpack/tests/svd_batch_test.cpp | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/mlpack/tests/svd_batch_test.cpp b/src/mlpack/tests/svd_batch_test.cpp
index 70d9a3a5d5..95db81e519 100644
--- a/src/mlpack/tests/svd_batch_test.cpp
+++ b/src/mlpack/tests/svd_batch_test.cpp
@@ -166,10 +166,8 @@ BOOST_AUTO_TEST_CASE(SVDBatchNegativeElementTest)
 
   arma::mat result = m1 * m2;
 
-  // 5% element-wise tolerance.
-  for (size_t i = 0; i < 3; i++)
-    for (size_t j = 0; j < 3; j++)
-      BOOST_REQUIRE_CLOSE(test(i, j), result(i, j), 5.0);
+  // 2% tolerance on the norm.
+  BOOST_REQUIRE_CLOSE(arma::norm(test, "fro"), arma::norm(result, "fro"), 2.0);
 }
 
 BOOST_AUTO_TEST_SUITE_END();