kannonier8/blender
1
0
Fork 0
forked from bartvdbraak/blender
Code Pull Requests Activity

Libmv: Cleanup, naming

Browse Source 
Initial bundle adjustment only supported OpenCV's radial distortion
model, so the cost functor was called after it.

Nowadays it supports more than this single model, so naming was a bit
wrong and misleading.
This commit is contained in:
Sergey Sharybin 2020-04-21 16:41:23 +02:00
parent b08e18ff36
commit 805c52b1fd
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
intern/libmv/libmv/simple_pipeline/bundle.cc
View File

@ -138,8 +138,8 @@ void ApplyDistortionModelUsingIntrinsicsBlock(
// //
// This functor can only be used for distortion models which have analytically // This functor can only be used for distortion models which have analytically
// defined Apply() function. // defined Apply() function.
struct OpenCVReprojectionErrorApplyIntrinsics { struct ReprojectionErrorApplyIntrinsics {
OpenCVReprojectionErrorApplyIntrinsics( ReprojectionErrorApplyIntrinsics(
const CameraIntrinsics *invariant_intrinsics, const CameraIntrinsics *invariant_intrinsics,
const double observed_distorted_x, const double observed_distorted_x,
const double observed_distorted_y, const double observed_distorted_y,
@ -421,7 +421,7 @@ void AddResidualBlockToProblem(const CameraIntrinsics *invariant_intrinsics,
double *camera_R_t, double *camera_R_t,
EuclideanPoint *point, EuclideanPoint *point,
ceres::Problem* problem) { ceres::Problem* problem) {
AddResidualBlockToProblemImpl<OpenCVReprojectionErrorApplyIntrinsics>( AddResidualBlockToProblemImpl<ReprojectionErrorApplyIntrinsics>(
invariant_intrinsics, invariant_intrinsics,
marker.x, marker.y, marker.x, marker.y,
marker_weight, marker_weight,