commit 2b7497025096a681d7f0351081f83293398d62ef
author Sameer Agarwal <sameeragarwal@google.com> Fri Apr 19 19:52:58 2013 -0700
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Sat Apr 20 02:54:45 2013 +0000
tree d6d2a770a30672653f1c5fda7a4f978797ff60e8
parent 9189f4ea4bb2d71ea7f6b9d9bd3290415aee323d

Fix a bounds error in the pre-ordering code.

Change-Id: I33c968bb075b60ad50374593302e08f42aeacf25

internal/ceres/solver_impl.cc

diff --git a/internal/ceres/solver_impl.cc b/internal/ceres/solver_impl.cc
index 8217484..43c0be6 100644
--- a/internal/ceres/solver_impl.cc
+++ b/internal/ceres/solver_impl.cc
@@ -1482,7 +1482,7 @@
   cholmod_sparse* block_jacobian_transpose =
       ss.CreateSparseMatrix(tsm_block_jacobian_transpose.get());
 
-  vector<int> ordering(program->NumResidualBlocks(), -1);
+  vector<int> ordering(program->NumParameterBlocks(), -1);
   ss.ApproximateMinimumDegreeOrdering(block_jacobian_transpose, &ordering[0]);
   ss.Free(block_jacobian_transpose);