From c91abb26441e0ed65cf3137a32c58c773f640234 Mon Sep 17 00:00:00 2001 From: Varun Agrawal Date: Fri, 24 Jan 2025 23:21:32 -0500 Subject: [PATCH] default arg for HybridSmoother constructor and clean up tests --- gtsam/hybrid/HybridSmoother.h | 2 +- gtsam/hybrid/tests/testHybridSmoother.cpp | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/gtsam/hybrid/HybridSmoother.h b/gtsam/hybrid/HybridSmoother.h index 96c8391b4..c3f022c62 100644 --- a/gtsam/hybrid/HybridSmoother.h +++ b/gtsam/hybrid/HybridSmoother.h @@ -40,7 +40,7 @@ class GTSAM_EXPORT HybridSmoother { * @param deadModeThreshold The threshold above which a mode gets assigned a * value and is considered "dead". 0.99 is a good starting value. */ - HybridSmoother(const std::optional deadModeThreshold) + HybridSmoother(const std::optional deadModeThreshold = {}) : deadModeThreshold_(deadModeThreshold) {} /** diff --git a/gtsam/hybrid/tests/testHybridSmoother.cpp b/gtsam/hybrid/tests/testHybridSmoother.cpp index 815e32560..3a0f376cc 100644 --- a/gtsam/hybrid/tests/testHybridSmoother.cpp +++ b/gtsam/hybrid/tests/testHybridSmoother.cpp @@ -194,8 +194,6 @@ TEST(HybridSmoother, DeadModeRemoval) { HybridGaussianFactorGraph linearized = *graph.linearize(initial); - // std::cout << "\n\n\nk" << std::endl; - // GTSAM_PRINT(linearized); smoother.update(linearized, maxNrLeaves); // Clear all the factors from the graph