saberbud/ceres-solver
1
0
Fork 0
mirror of https://github.com/ceres-solver/ceres-solver.git synced 2026-08-29 08:34:37 +08:00
Code Issues Packages Projects Releases Wiki Activity

Update ExpectArraysClose to use ExpectClose instead of EXPECT_NEAR

Browse Source 
The documentation for ExpectArraysClose and its implementation
did not match.

This change makes the polynomial_test not fail on 64bit AMD builds.

Thanks to Phillip Huebner for reporting this.

Change-Id: I503f2d3317a28d5885a34f8bdbccd49d20ae9ba2
This commit is contained in:
Sameer Agarwal
2016-09-01 00:05:09 -07:00
parent 2fd39fcecb
commit 8c4623c63a
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
internal/ceres/test_util.cc
+4 -1
View File
@@ -117,7 +117,10 @@ void ExpectArraysClose(int n,
CHECK(q);
for (int i = 0; i < n; ++i) {
EXPECT_NEAR(p[i], q[i], tol) << "i=" << i;
EXPECT_TRUE(ExpectClose(p[i], q[i], tol))
<< "p[" << i << "]" << p[i] << " "
<< "q[" << i << "]" << q[i] << " "
<< "tol: " << tol;
}
}