Skip to content

Commit

Permalink
bugfix on parallel & linear solver change
Browse files Browse the repository at this point in the history
  • Loading branch information
amy-tabb committed Dec 16, 2020
1 parent 51831dd commit 6c4fca1
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions src/camera_calibration.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1152,7 +1152,7 @@ void CameraCali::FindCornersArucoCharuco(const string& write_dir, bool verbose,
two_d_point_coordinates_dense.resize(number_images, twod);
points_present.resize(number_images, vector<bool>(number_squares, false));

#pragma omp parallel for private(b_color, global_index)
#pragma omp parallel for private(b_color, global_index, filename)
for (int i = 0; i < number_images; i++){

vector< vector<cv::Point2f> > charuco_corners_all;
Expand Down Expand Up @@ -1309,7 +1309,7 @@ void CameraCali::FindCornersCharuco(const string& write_dir, bool write_internal

int internal_found_counter = 0;

#pragma omp parallel for private(b_color, global_index)
#pragma omp parallel for private(b_color, global_index, filename)
for (int i = 0; i < number_images; i++){

vector< int > ids;
Expand Down
2 changes: 1 addition & 1 deletion src/solving_structure.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -486,7 +486,7 @@ void CeresProblemClass::SolveWriteBackToMC(MCcali& MC, std::ofstream& out, int i


Solver::Options options;
options.linear_solver_type = ceres::DENSE_QR;
options.linear_solver_type = ceres::DENSE_SCHUR;

options.minimizer_progress_to_stdout = output_to_terminal;
options.num_threads = omp_get_max_threads();
Expand Down

0 comments on commit 6c4fca1

Please sign in to comment.