diff options
author | Sergey Sharybin <sergey.vfx@gmail.com> | 2016-12-14 12:44:57 +0300 |
---|---|---|
committer | Sergey Sharybin <sergey.vfx@gmail.com> | 2016-12-14 12:46:20 +0300 |
commit | fbcd51aef86a69fb99ae5cba51da6401f6c3dca4 (patch) | |
tree | 1a7496648e6e018990f30a6cd63f1eaaa2964e9b | |
parent | c21a5139e1f263c0c7acd80ec49d5e3ecbff8456 (diff) |
Fix T50243: libmv_panography_test is broken
There was fully wrong logic in comparison: was actually accessing memory
past the array boundary. Run test manually and the figure seems correct
to me now.
Spotted by @LazyDodo, thanks!
-rw-r--r-- | intern/libmv/libmv/multiview/panography_test.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/intern/libmv/libmv/multiview/panography_test.cc b/intern/libmv/libmv/multiview/panography_test.cc index f6faf0f6022..96d52acfc3c 100644 --- a/intern/libmv/libmv/multiview/panography_test.cc +++ b/intern/libmv/libmv/multiview/panography_test.cc @@ -48,7 +48,7 @@ TEST(Panography, PrintSomeSharedFocalEstimationValues) { // Assert we found a valid solution. EXPECT_EQ(1, fs.size()); - EXPECT_NEAR(1.01667, fs[1], 1e-3); + EXPECT_NEAR(3.47194, fs[0], 1e-3); } TEST(Panography, GetR_FixedCameraCenterWithIdentity) { |