From b0ad350a20a410aa2658839a7707a910d7129bc6 Mon Sep 17 00:00:00 2001
From: Varun Agrawal <varagrawal@gmail.com>
Date: Sat, 7 Dec 2024 18:22:30 -0500
Subject: [PATCH] add note about toDecisionTreeFactor

---
 gtsam/discrete/TableFactor.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/gtsam/discrete/TableFactor.cpp b/gtsam/discrete/TableFactor.cpp
index b867fa916..53131616d 100644
--- a/gtsam/discrete/TableFactor.cpp
+++ b/gtsam/discrete/TableFactor.cpp
@@ -182,14 +182,13 @@ DiscreteFactor::shared_ptr TableFactor::operator*(
 
 /* ************************************************************************ */
 DecisionTreeFactor TableFactor::toDecisionTreeFactor() const {
-  gttic_(toDecisionTreeFactor);
   DiscreteKeys dkeys = discreteKeys();
   std::vector<double> table;
   for (auto i = 0; i < sparse_table_.size(); i++) {
     table.push_back(sparse_table_.coeff(i));
   }
-  gttoc_(toDecisionTreeFactor);
   gttic_(toDecisionTreeFactor_Constructor);
+  // NOTE(Varun): This constructor is really expensive!!
   DecisionTreeFactor f(dkeys, table);
   gttoc_(toDecisionTreeFactor_Constructor);
   return f;