Skip to content

Conversation

dellaert
Copy link
Member

Stubs for @giuliodelama

@giuliodelama please continue to work on ths branch, but only push when you need review or want to run CI.

Documentation
Simplify
Force MODULE
Fix back-slashes
Upgrade cmake req
Well-documented gemini solution
Going in circles...
Give gemini a chance
yet another try
More windows/boost fixes
Yet another attempt
Another attempt, aided by GPT-5
Try fixing boost on windows
@dellaert dellaert force-pushed the feature/GalileanImuFactor branch from 182580b to 02b578e Compare August 23, 2025 21:15
TEST_PIM(CombinedImuFactor, PreintegratedMeasurements) {
// Linearization point
Bias bias(Vector3(0, 0, 0), Vector3(0, 0, 0)); ///< Current estimate of acceleration and angular rate biases
Bias bias(

Choose a reason for hiding this comment

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

???

- no IMU biases
- preintegration covariance propagation is for RI error
preintMatrix_ = preintMatrix_.compose(wExp);

// Propagate covariance - from eq. (35) in the paper
// IMPORTANT: the covariance propagation is based on the Right-Invariant (RI)
Copy link
Member Author

Choose a reason for hiding this comment

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

Are these outdated now? I think you switched to LI?

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.

3 participants