Commit: 551265424b3b41bd87943215703715168d3212a2
Author: Sergey Sharybin
Date: Wed Dec 14 10:44:57 2016 +0100
Branches: blender-v2.78c-release
https://developer.blender.org/rB551265424b3b41bd87943215703715168d3212a2
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!
===================================================================
M intern/libmv/libmv/multiview/panography_test.cc
===================================================================
diff --git a/intern/libmv/libmv/multiview/panography_test.cc
b/intern/libmv/libmv/multiview/panography_test.cc
index f6faf0f602..96d52acfc3 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) {
_______________________________________________
Bf-blender-cvs mailing list
[email protected]
https://lists.blender.org/mailman/listinfo/bf-blender-cvs
