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

[Feature-Request] Camera - IMU synced in Sync Node #930

Closed
michaelnguyen11 opened this issue Nov 30, 2023 · 2 comments
Closed

[Feature-Request] Camera - IMU synced in Sync Node #930

michaelnguyen11 opened this issue Nov 30, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@michaelnguyen11
Copy link

Start with the why:

Camera-IMU messages synchronization are very important in VIO/SLAM development.

Move to the what:

A pull request #917 implemented a Sync Node that synchronized timestamp of camera left/right/center on device then send to host for display. It would be nice if the IMU data can be synchronized with camera data.

Move to the how:

I don't have any idea to implement that yet.

Hi @asahtik ,

Know that you are the author of pull request #917 . Do you have any plans to support the IMU data to be synchronized with camera data through the Sync Node and Message Groups? Thank you in advance!

Best regards,
Hiep Nguyen

@michaelnguyen11 michaelnguyen11 added the enhancement New feature or request label Nov 30, 2023
@asahtik
Copy link
Contributor

asahtik commented Nov 30, 2023

Hi @michaelnguyen11, great question. Currently the Sync node synchronizes a group of messages based on their device timestamps. The issue with IMUData is that it groups multiple packets (with timestamps) together and so it does not have its own timestamp - this requires some special handling that is not yet implemented. However, we are planning on supporting syncing IMU packets through the Sync node.

@michaelnguyen11
Copy link
Author

Hi @asahtik ,

Thank you for your information. Hope to test the syncing Camera-IMU in the Sync node soon.

jakgra pushed a commit that referenced this issue Mar 20, 2024
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

2 participants