diff options
author | Sergey Sharybin <sergey@blender.org> | 2020-10-12 11:39:04 +0300 |
---|---|---|
committer | Sergey Sharybin <sergey@blender.org> | 2020-10-12 16:12:17 +0300 |
commit | 86138d88f6b41790ff043de7625410afb08a56e2 (patch) | |
tree | fd1c885d43fbd25e849e0004ff68f59993705e5c | |
parent | df5b65811e4ebbe02524c94defdf7be4ae0275ba (diff) |
Revert "Fix critical lens distortion bug in libmv after rB3a7d62cd1f5e."
This reverts commit 7e836bde11ce6521953c9f246cacd442a3ef6c0e.
Reverting the incomplete workaround, due to the following reasoning:
- There should be no dependency between CameraIntrinsics and the bundler
code. This violates intended abstraction of the intrinsics class.
- The fix was not complete, or wrong. Even in the world where there is
a requirement to the parameters storage size this should have applied
to all distortion models, including Divisions, Nuke.
Proper fix will be committed next.
-rw-r--r-- | intern/libmv/libmv/simple_pipeline/bundle.cc | 4 | ||||
-rw-r--r-- | intern/libmv/libmv/simple_pipeline/camera_intrinsics.h | 1 |
2 files changed, 0 insertions, 5 deletions
diff --git a/intern/libmv/libmv/simple_pipeline/bundle.cc b/intern/libmv/libmv/simple_pipeline/bundle.cc index 4e76ae1f473..c055846318a 100644 --- a/intern/libmv/libmv/simple_pipeline/bundle.cc +++ b/intern/libmv/libmv/simple_pipeline/bundle.cc @@ -47,10 +47,6 @@ enum { OFFSET_PRINCIPAL_POINT_Y, // Distortion model coefficients. - // WARNING: camera intrinsics are not required to define/use all of those, - // but they have to define all of them up to the last one they need, in that - // exact same order. See e.g. how `PolynomialCameraIntrinsics` needs to define - // `OFFSET_K4`, even though it does not use it. OFFSET_K1, OFFSET_K2, OFFSET_K3, diff --git a/intern/libmv/libmv/simple_pipeline/camera_intrinsics.h b/intern/libmv/libmv/simple_pipeline/camera_intrinsics.h index 60ad1b5b8de..cf0bdb76ccb 100644 --- a/intern/libmv/libmv/simple_pipeline/camera_intrinsics.h +++ b/intern/libmv/libmv/simple_pipeline/camera_intrinsics.h @@ -281,7 +281,6 @@ class PolynomialCameraIntrinsics : public CameraIntrinsics { OFFSET_K1, OFFSET_K2, OFFSET_K3, - OFFSET_K4, // Unused, needed to map properly with all deform parameters defined in bundle.cc. OFFSET_P1, OFFSET_P2, |