Update Summary::FullReport to report dogleg type. Change-Id: I0b4be8d7486c1c4b36b299693b3fe8b0d3426537
diff --git a/include/ceres/solver.h b/include/ceres/solver.h index 16c7387..31d5e8d 100644 --- a/include/ceres/solver.h +++ b/include/ceres/solver.h
@@ -511,6 +511,7 @@ OrderingType ordering_type; TrustRegionStrategyType trust_region_strategy_type; + DoglegType dogleg_type; SparseLinearAlgebraLibraryType sparse_linear_algebra_library; };
diff --git a/internal/ceres/solver.cc b/internal/ceres/solver.cc index ee98392..66ca932 100644 --- a/internal/ceres/solver.cc +++ b/internal/ceres/solver.cc
@@ -197,9 +197,18 @@ sparse_linear_algebra_library)); } - internal::StringAppendF(&report, "Trust Region Strategy %19s\n", + internal::StringAppendF(&report, "Trust Region Strategy %19s", TrustRegionStrategyTypeToString( trust_region_strategy_type)); + if (trust_region_strategy_type == DOGLEG) { + if (dogleg_type == TRADITIONAL_DOGLEG) { + internal::StringAppendF(&report, " (TRADITIONAL)"); + } else { + internal::StringAppendF(&report, " (SUBSPACE)"); + } + } + internal::StringAppendF(&report, "\n"); + if (termination_type == DID_NOT_RUN) { CHECK(!error.empty())
diff --git a/internal/ceres/solver_impl.cc b/internal/ceres/solver_impl.cc index 072f8bb..2802a75 100644 --- a/internal/ceres/solver_impl.cc +++ b/internal/ceres/solver_impl.cc
@@ -238,6 +238,7 @@ summary->sparse_linear_algebra_library = options.sparse_linear_algebra_library; summary->trust_region_strategy_type = options.trust_region_strategy_type; + summary->dogleg_type = options.dogleg_type; // Evaluate the initial cost, residual vector and the jacobian // matrix if requested by the user. The initial cost needs to be