12345qiupeng
/
gtsam
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

address review comments

release/4.3a0
Varun Agrawal 2022-06-10 04:24:13 -04:00
parent 09d512ae48
commit 7496f2f43a
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
gtsam/hybrid/HybridGaussianFactorGraph.h
View File

@ -134,8 +134,7 @@ class GTSAM_EXPORT HybridGaussianFactorGraph
template <typename FACTOR> template <typename FACTOR>
IsGaussian<FACTOR> push_gaussian( IsGaussian<FACTOR> push_gaussian(
const boost::shared_ptr<FACTOR>& gaussianFactor) { const boost::shared_ptr<FACTOR>& gaussianFactor) {
Base::Base::push_back( Base::push_back(boost::make_shared<HybridGaussianFactor>(gaussianFactor));
boost::make_shared<HybridGaussianFactor>(gaussianFactor));
} }
/// Construct a factor and add (shared pointer to it) to factor graph. /// Construct a factor and add (shared pointer to it) to factor graph.

1
gtsam/hybrid/tests/testGaussianHybridFactorGraph.cpp
View File

@ -273,6 +273,7 @@ TEST(HybridGaussianFactorGraph, eliminateFullMultifrontalTwoClique) {
// GTSAM_PRINT(*hbt); // GTSAM_PRINT(*hbt);
// GTSAM_PRINT(*remaining); // GTSAM_PRINT(*remaining);
// hbt->marginalFactor(X(1))->print("HBT: ");
/* /*
(Fan) Explanation: the Junction tree will need to reeliminate to get to the (Fan) Explanation: the Junction tree will need to reeliminate to get to the
marginal on X(1), which is not possible because it involves eliminating marginal on X(1), which is not possible because it involves eliminating