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: imu offset compensation service #88

Open
john-maidbot opened this issue Jan 11, 2025 · 2 comments
Open

Feature Request: imu offset compensation service #88

john-maidbot opened this issue Jan 11, 2025 · 2 comments

Comments

@john-maidbot
Copy link

john-maidbot commented Jan 11, 2025

I find that the imu in my orbbec sensor is accurate but often has an initial offset in angular velocity and linear acceleration that needs to be corrected. I see this behavior on both femto and gemini-335 (but they have different offsets)

i see that you already correct for bias accumulation in the ros2 wrapper, but it would be nice to also have an option to correct for this initial offset (eg place sensor on level surface and call a ros2 service. Thats what i am doing on my end atm, but it would be nice if this was in the orbbec node. Plus i figure others would benefit from this too). Have you considered adding such a feature? Or does this offset perhaps suggest something is wrong with the createIMUInfo()?

thank you for your time

@jjiszjj
Copy link
Collaborator

jjiszjj commented Feb 19, 2025

@john-maidbot Hello,thank you very much for your suggestion.From what I understand from the text you said, do you mean to add a new service function to recalibrate the initial position of imu?

@john-maidbot
Copy link
Author

john-maidbot commented Feb 19, 2025

Thank you for your reply.

The intention of the service would be to recalibrate the acceleration and angular velocity bias offsets of the imu while level and at rest. Not the position.

Ie. While at rest and level, imu angular velocity should be nearly (0,0,0) rad/s and acceleration should be nearly (0,0,-9.81) m/s^2 (assuming imu z axis is aligned with gravity for simplicity here). And currently that is often not the case when i start the cameras

I can share some sensor data if that would be helpful?

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

No branches or pull requests

2 participants