Skip to content

chore: scaffold navigation estimation core - #751

Draft
ApatShe wants to merge 3 commits into
728-task-odom-estimatorfrom
728-scaffold-navigation-core
Draft

ApatShe wants to merge 3 commits into
728-task-odom-estimatorfrom
728-scaffold-navigation-core

Conversation

@ApatShe

@ApatShe ApatShe commented Oct 17, 2026 •

Copy link
Copy Markdown

Add empty source and header scaffolding for the estimation core:

  • preintegration: accumulate IMU samples into relative motion and uncertainty for prediction and IMU constraints.
  • factor_builders: construct built-in and expression factors from sensor measurements and noise models for iSAM2.
  • estimation: initialize navigation, associate buffered measurements with states, and coordinate prediction and iSAM2 updates.

Design references:

  • Factor Graphs for Robot Perception: §§2.2–2.3 (measurement constraints), §§5.4.2–5.4.3 (incremental updates and iSAM2), and §5.3.2 (fixed-lag estimation).
  • iSAM2: §2 (estimation problem), §4.3 and Algorithm 8

Design references:
- Factor Graphs for Robot Perception — Dellaert and Kaess: https://www.cs.cmu.edu/~kaess/pub/Dellaert17fnt.pdf, §§2.2–2.3: measurement likelihoods, weighted residual costs and linearization.
- iSAM2: Incremental Smoothing and Mapping Using the Bayes Tree — Kaess et al.: https://www.cs.cmu.edu/~kaess/pub/Kaess12ijrr.pdf, §2: the nonlinear factor-graph estimation problem supplied to the solver.
Design references:
- Factor Graphs for Robot Perception — Dellaert and Kaess: https://www.cs.cmu.edu/~kaess/pub/Dellaert17fnt.pdf, §§5.4.2–5.4.3: incremental Bayes-tree updates and iSAM2. §5.3.2 provides the background for the intended fixed-lag estimation window.
- iSAM2: Incremental Smoothing and Mapping Using the Bayes Tree — Kaess et al.: https://www.cs.cmu.edu/~kaess/pub/Kaess12ijrr.pdf, §4.3 and Algorithm 8: the incremental update cycle. Section and algorithm numbering refer to this linked author manuscript.
@ApatShe
ApatShe force-pushed the 728-scaffold-navigation-core branch from b2d9a1a to beb45b3 Compare October 17, 2026 16:06
@ApatShe
ApatShe requested a balanced review from Copilot October 17, 2026 16:11

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.

@ApatShe
ApatShe requested review from chrstrom and jorgenfj October 17, 2026 16:11
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