blender
/
blender
128
396
Fork 568
Code Issues 6.2k Pull Requests 751 Projects 19 Wiki Activity

Libmv: Cleanup, naming 

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
// defined Apply() function.
struct OpenCVReprojectionErrorApplyIntrinsics {
OpenCVReprojectionErrorApplyIntrinsics(
struct ReprojectionErrorApplyIntrinsics {
ReprojectionErrorApplyIntrinsics(
const CameraIntrinsics *invariant_intrinsics,
const double observed_distorted_x,
const double observed_distorted_y,
@ -421,7 +421,7 @@ void AddResidualBlockToProblem(const CameraIntrinsics *invariant_intrinsics,
double *camera_R_t,
EuclideanPoint *point,
ceres::Problem* problem) {
AddResidualBlockToProblemImpl<OpenCVReprojectionErrorApplyIntrinsics>(
AddResidualBlockToProblemImpl<ReprojectionErrorApplyIntrinsics>(
invariant_intrinsics,
marker.x, marker.y,
marker_weight,