Hybrid Updates by varunagrawal · Pull Request #1306 · borglab/gtsam · GitHub
Skip to content

Hybrid Updates - #1306

Merged
varunagrawal merged 28 commits into
developfrom
varun/test-hybrid-estimation
Oct 14, 2022
Merged

Hybrid Updates#1306
varunagrawal merged 28 commits into
developfrom
varun/test-hybrid-estimation

Conversation

@varunagrawal

Copy link
Copy Markdown
Contributor
  1. Move pruner functor to GaussianMixture so we remove duplication and use the same API for both the HybridBayesTree and the HybridBayesNet.
  2. Fix issue where pruning doesn't happen for subset of discrete keys (based on @ProfFan's idea).
  3. Various formatting and niggles (GaussianMixtureFactor.cpp, HybridFactor.cpp, HybridGaussianFactorGraph.cpp, HybridGaussianISAM.cpp).
  4. Add key formatter to HybridNonlinearISAM::print.
  5. Fix compile time warnings from testDiscreteBayesTree.cpp.

@varunagrawal varunagrawal self-assigned this Oct 13, 2022
Comment thread gtsam/hybrid/GaussianMixture.cpp
@varunagrawal
varunagrawal merged commit cd69c51 into develop Oct 14, 2022
@varunagrawal
varunagrawal deleted the varun/test-hybrid-estimation branch October 14, 2022 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants