Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IMU integration #235

Open
11 tasks
ymd-stella opened this issue Dec 23, 2021 · 5 comments
Open
11 tasks

IMU integration #235

ymd-stella opened this issue Dec 23, 2021 · 5 comments
Labels
enhancement New feature or request

Comments

@ymd-stella
Copy link
Contributor

ymd-stella commented Dec 23, 2021

Tasks

  • Consider how to do unit testing or benchmarking
    • unit testing of IMU preprocessings (compute_velocity, compute_gravity_dir, preintegration)
    • unit testing of edges for visual inertial
    • unit testing (or benchmarking) of globalBA, localBA, prediction, loop closure
  • Implementation features and unit tests
    • globalBA
    • localBA
    • prediction
    • loop closure
  • Stabilize the scale and gravity direction initialization if possible.
    • Make the keyframe insertion strategy customizable.
    • Consider a default keyframe insertion strategy when using IMU.
  • Investigate how to calibrate the camera and IMU.
    • (The pinhole camera model is supported by Kalibr.)
  • Exploring data sets

Implementation considerations

  • Consider how to implement without losing readability and usability.
  • Do not optimize too early. Only adopt methods that have been shown to be effective by benchmarking.

wip branch

wip/gba is an experimental implementation, but it has some issues.

  • The edge implementation is untested and suspicious.
  • It pollutes the existing global BA.
  • It is very unstable. The result will change each time you run it. In particular, the calculation of the direction of gravity is unstable.
  • There seems to be a difference in the optimal strategy for keyframe insertion: compared to Visual SLAM, we needed to insert more keyframes for scale and gravity direction initialization.
  • My impression from the experiments so far is that the time taken for initialization is too long. In some applications, it may be necessary to move the camera randomly until it is initialized.

references

@SeungRyeol
Copy link

Hello @ymd-stella. Thank you for your continuous code update.
If I use the code of branch "feat/imu", can I use visual-inertial SLAM like ORB-SLAM3?

@ymd-stella
Copy link
Contributor Author

No.

@special00 special00 mentioned this issue Mar 24, 2023
@YznMur
Copy link

YznMur commented May 19, 2023

any update?

@ymd-stella
Copy link
Contributor Author

None.

@adamanov
Copy link

any updates regarding on IMU integration?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants