commit f55780063620e7a3dcfe7e018d6488bf6a5b29da
author Sameer Agarwal <sameeragarwal@google.com> Wed Jan 08 10:43:31 2014 -0800
committer Sameer Agarwal <sameeragarwal@google.com> Wed Jan 08 10:45:56 2014 -0800
tree f1cd54a302adfa5e7c96e29b455a4d11a270e4d2
parent 85561eee951c91e578984c6d3eecf0073acabb64

Reduce logging verbosity.

When user specifies Solver::Options::logging_type = SILENT,
ensure that the minimizer does not log anything.

Change-Id: I94e34dae504881ab36d4a66e6adb7a19a227363e

internal/ceres/minimizer.h

diff --git a/internal/ceres/minimizer.h b/internal/ceres/minimizer.h
index 3d9da99..ee77726 100644
--- a/internal/ceres/minimizer.h
+++ b/internal/ceres/minimizer.h
@@ -107,7 +107,7 @@
           options.line_search_sufficient_curvature_decrease;
       max_line_search_step_expansion =
           options.max_line_search_step_expansion;
-      is_silent = false;
+      is_silent = (options.logging_type == SILENT);
       evaluator = NULL;
       trust_region_strategy = NULL;
       jacobian = NULL;

internal/ceres/solver_impl.cc

diff --git a/internal/ceres/solver_impl.cc b/internal/ceres/solver_impl.cc
index a26ef9f..cf0e251 100644
--- a/internal/ceres/solver_impl.cc
+++ b/internal/ceres/solver_impl.cc
@@ -484,7 +484,7 @@
         "Terminating: Function tolerance reached. "
         "No non-constant parameter blocks found.";
     summary->termination_type = CONVERGENCE;
-    VLOG(1) << summary->message;
+    VLOG_IF(1, options.logging_type != SILENT) << summary->message;
 
     summary->initial_cost = summary->fixed_cost;
     summary->final_cost = summary->fixed_cost;
@@ -804,7 +804,7 @@
         "Terminating: Function tolerance reached. "
         "No non-constant parameter blocks found.";
     summary->termination_type = CONVERGENCE;
-    VLOG(1) << summary->message;
+    VLOG_IF(1, options.logging_type != SILENT) << summary->message;
 
     const double post_process_start_time = WallTimeInSeconds();
     SetSummaryFinalCost(summary);