Actually the Colamd wrap is fine

release/4.3a0
p-zach 2025-04-16 00:02:07 -04:00
parent 3db7920f65
commit 4c4bdd4ba7
1 changed files with 2 additions and 4 deletions

View File

@ -123,10 +123,8 @@ class Ordering {
FACTOR_GRAPH = {gtsam::NonlinearFactorGraph, gtsam::DiscreteFactorGraph,
gtsam::SymbolicFactorGraph, gtsam::GaussianFactorGraph, gtsam::HybridGaussianFactorGraph}>
// Can't have both of these Colamd implementations; the second overrides the first.
// Python wrapper is for ease of use, so choose the one that is more commonly used.
static gtsam::Ordering Colamd(const FACTOR_GRAPH& graph);
// static gtsam::Ordering Colamd(const gtsam::VariableIndex& variableIndex);
static gtsam::Ordering Colamd(const gtsam::VariableIndex& variableIndex);
template <
FACTOR_GRAPH = {gtsam::NonlinearFactorGraph, gtsam::DiscreteFactorGraph,