saberbud/ceres-solver
1
0
Fork 0
mirror of https://github.com/ceres-solver/ceres-solver.git synced 2026-08-29 16:40:38 +08:00
Code Issues Packages Projects Releases Wiki Activity

Remove Solver::Options::use_postordering

Browse Source 
This was an ill-advised and complicated to interpret option
which offers nothing particularly useful.

Change-Id: Ia7741ed62ef977c96fa52299a884e404bee659ac
This commit is contained in:
Sameer Agarwal
2022-05-19 13:27:59 -07:00
parent 30b4d5df35
commit 8ba8fbb173
5 changed files with 5 additions and 48 deletions
Download Patch File Download Diff File Expand all files Collapse all files
include/ceres/solver.h
-10
View File
@@ -588,16 +588,6 @@ class CERES_EXPORT Solver {
// Jacobian matrix and generally speaking, there is no performance
// penalty for doing so.
// TODO(sameeragarwal): Remove this option. It is too obscure and
// there is no clear way of figuring out when this is a useful
// thing to do.
//
// In some rare cases, it is worth using a more complicated
// reordering algorithm which has slightly better runtime
// performance at the expense of an extra copy of the Jacobian
// matrix. Setting use_postordering to true enables this tradeoff.
bool use_postordering = false;
// Some non-linear least squares problems are symbolically dense but
// numerically sparse. i.e. at any given state only a small number
// of jacobian entries are non-zero, but the position and number of