From 198efe23c7cc588acc52536a2268ec7877230296 Mon Sep 17 00:00:00 2001 From: p-zach Date: Tue, 15 Apr 2025 18:16:51 -0400 Subject: [PATCH] Remove COLAMD varindex overload --- gtsam/inference/inference.i | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/gtsam/inference/inference.i b/gtsam/inference/inference.i index 5084355a8..13c4c9853 100644 --- a/gtsam/inference/inference.i +++ b/gtsam/inference/inference.i @@ -122,8 +122,11 @@ class Ordering { template < 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,