commit 1e2892009e591804df6286caebd5c960e7e3b099
author Sameer Agarwal <sameeragarwal@google.com> Tue Aug 21 18:00:54 2012 -0700
committer ceres-solver code review <noreply-gerritcodereview@google.com> Wed Aug 22 08:50:17 2012 -0700
tree ccc3cac6092014be9bbbac1ce57b04d03f699615
parent 295ade1122a86b83e1ea605d5ca394f315874717

Update Summary::FullReport to report dogleg type.

Change-Id: I0b4be8d7486c1c4b36b299693b3fe8b0d3426537

internal/ceres/solver.cc

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())

internal/ceres/solver_impl.cc

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