switch from DiscreteKeys back to std::set<DiscreteKey>

borglab · Jan 16, 2023 · 5e1de8c · 5e1de8c
1 parent 64743ef
commit 5e1de8c
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/gtsam/hybrid/HybridFactorGraph.cpp b/gtsam/hybrid/HybridFactorGraph.cpp
@@ -25,7 +25,7 @@
 namespace gtsam {
 
 /* ************************************************************************* */
-DiscreteKeys HybridFactorGraph::discreteKeys() const {
+std::set<DiscreteKey> HybridFactorGraph::discreteKeys() const {
   std::set<DiscreteKey> keys;
   for (auto& factor : factors_) {
     if (auto p = boost::dynamic_pointer_cast<DecisionTreeFactor>(factor)) {
@@ -39,14 +39,14 @@ DiscreteKeys HybridFactorGraph::discreteKeys() const {
       }
     }
   }
-  return DiscreteKeys(keys.begin(), keys.end());
+  return keys;
 }
 
 /* ************************************************************************* */
 KeySet HybridFactorGraph::discreteKeySet() const {
   KeySet keys;
-  DiscreteKeys key_vector = discreteKeys();
-  std::transform(key_vector.begin(), key_vector.end(),
+  std::set<DiscreteKey> key_set = discreteKeys();
+  std::transform(key_set.begin(), key_set.end(),
                  std::inserter(keys, keys.begin()),
                  [](const DiscreteKey& k) { return k.first; });
   return keys;

diff --git a/gtsam/hybrid/HybridFactorGraph.h b/gtsam/hybrid/HybridFactorGraph.h
@@ -65,7 +65,7 @@ class HybridFactorGraph : public FactorGraph<Factor> {
   /// @{
 
   /// Get all the discrete keys in the factor graph.
-  DiscreteKeys discreteKeys() const;
+  std::set<DiscreteKey> discreteKeys() const;
 
   /// Get all the discrete keys in the factor graph, as a set.
   KeySet discreteKeySet() const;