From e1bcc6e0f51512f43aa7bfb7b0d62f7ac1d0cd4b Mon Sep 17 00:00:00 2001
From: Sameer Agarwal <sameeragarwal@google.com>
Date: Wed, 18 May 2016 07:52:48 -0700
Subject: [PATCH] Add additional logging for analyzing orderings

Change-Id: Ic68d2959db35254e2895f11294fb25de4d4b8a81
---
 internal/ceres/reorder_program.cc |  5 +++++
 internal/ceres/solver.cc          | 18 ++++++++----------
 2 files changed, 13 insertions(+), 10 deletions(-)

diff --git a/internal/ceres/reorder_program.cc b/internal/ceres/reorder_program.cc
index d0e8f32b3..a7c371075 100644
--- a/internal/ceres/reorder_program.cc
+++ b/internal/ceres/reorder_program.cc
@@ -142,6 +142,11 @@ void OrderingForSparseNormalCholeskyUsingSuiteSparse(
                                                    ordering);
   }
 
+  VLOG(2) << "Block ordering stats: "
+          << " flops: " << ss.mutable_cc()->fl
+          << " lnz  : " << ss.mutable_cc()->lnz
+          << " anz  : " << ss.mutable_cc()->anz;
+
   ss.Free(block_jacobian_transpose);
 #endif  // CERES_NO_SUITESPARSE
 }
diff --git a/internal/ceres/solver.cc b/internal/ceres/solver.cc
index 5c784f81d..21d3c6869 100644
--- a/internal/ceres/solver.cc
+++ b/internal/ceres/solver.cc
@@ -698,16 +698,14 @@ string Solver::Summary::FullReport() const {
                   num_linear_solver_threads_given,
                   num_linear_solver_threads_used);
 
-    if (IsSchurType(linear_solver_type_used)) {
-      string given;
-      StringifyOrdering(linear_solver_ordering_given, &given);
-      string used;
-      StringifyOrdering(linear_solver_ordering_used, &used);
-      StringAppendF(&report,
-                    "Linear solver ordering %22s %24s\n",
-                    given.c_str(),
-                    used.c_str());
-    }
+    string given;
+    StringifyOrdering(linear_solver_ordering_given, &given);
+    string used;
+    StringifyOrdering(linear_solver_ordering_used, &used);
+    StringAppendF(&report,
+                  "Linear solver ordering %22s %24s\n",
+                  given.c_str(),
+                  used.c_str());
 
     if (inner_iterations_given) {
       StringAppendF(&report,