commit 603277d34392eeb14cd194f7e22b0d6db247ee1e
author Sameer Agarwal <sameeragarwal@google.com> Tue Jun 10 10:00:45 2014 -0700
committer Sameer Agarwal <sameeragarwal@google.com> Tue Jun 10 10:07:53 2014 -0700
tree fcbd9baf7d9774325f395e2df4c39ac52fb87eff
parent e9fcf3decba3352a33857265b8dbf820fb1a43e6

Add check for dynamic sparsity in Solver::Options::IsValid.

Dynamic sparsity should only be used with SPARSE_NORMAL_CHOLESKY.

Change-Id: I8dd33c05ec0337b3c13b547ecffe3aefc3e5ab12

internal/ceres/solver.cc

diff --git a/internal/ceres/solver.cc b/internal/ceres/solver.cc
index 8f19886..e99ddc9 100644
--- a/internal/ceres/solver.cc
+++ b/internal/ceres/solver.cc
@@ -216,6 +216,12 @@
     return false;
   }
 
+  if (options.dynamic_sparsity &&
+      options.linear_solver_type != SPARSE_NORMAL_CHOLESKY) {
+    *error = "Dynamic sparsity is only supported with SPARSE_NORMAL_CHOLESKY.";
+    return false;
+  }
+
   return true;
 }