Made FactorGraph equals protected since it should be overridden

gtsam/inference/FactorGraphUnordered.h

@@ -155,12 +155,14 @@ namespace gtsam {
 
     /** print out graph */
     void print(const std::string& s = "FactorGraph",
-        const KeyFormatter& formatter = DefaultKeyFormatter) const;
+      const KeyFormatter& formatter = DefaultKeyFormatter) const;
 
+  protected:
     /** Check equality */
     bool equals(const This& fg, double tol = 1e-9) const;
-
     /// @}
+
+  public:
     /// @name Standard Interface
     /// @{
 

gtsam/symbolic/SymbolicBayesNetUnordered.cpp

@@ -22,7 +22,9 @@
 namespace gtsam {
 
   /* ************************************************************************* */
-  void SymbolicBayesNetUnordered::noop() const {
+  bool SymbolicBayesNetUnordered::equals(const This& bn, double tol = 1e-9) const
+  {
+    return Base::equals(bn, tol);
   }
 
 

gtsam/symbolic/SymbolicBayesNetUnordered.h

@@ -18,9 +18,9 @@
 
 #pragma once
 
-#include <gtsam/base/types.h>
-#include <gtsam/inference/BayesNetUnordered.h>
 #include <gtsam/symbolic/SymbolicConditionalUnordered.h>
+#include <gtsam/inference/BayesNetUnordered.h>
+#include <gtsam/base/types.h>
 
 namespace gtsam {
 
@@ -47,13 +47,19 @@ namespace gtsam {
     SymbolicBayesNetUnordered(ITERATOR firstConditional, ITERATOR lastConditional) : Base(firstConditional, lastConditional) {}
     
     /// @}
+
+    /// @name Testable
+    /// @{
+    
+    /** Check equality */
+    bool equals(const This& bn, double tol = 1e-9) const;
+
+    /// @}
+    
     /// @name Standard Interface
     /// @{
     
     /// @}
-
-  private:
-    void noop() const; // Function defined in cpp file so that compiler instantiates the base class
   };
 
 } // namespace gtsam