-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
Use gyro/accel/mag raw values for SCALED_IMU mavlink msg #8517
Conversation
f18ce46
to
b062b6e
Compare
Just curious - what's wrong with the values in |
@mhkabir For two reasons:
We've tried using |
this looks like a valid short-term solution but the changes to all sensors are incomplete. I'm still pondering if this is the right incremental step or if we need something different (unfiltered, but correctly scaled values) across the board and rather should be doing that. The current scaling does not have enough resolution to not introduce integration errors, which means its suboptimal for VIO and not viable long-term. |
@LorenzMeier I added the missing drivers (hopefully, I got them all). I was wondering about the resolution as well. But what what do you mean with integration errors? Rounding errors? The thing is, it works reliably with VISLAM and therefore I'm not too worried. |
@ChristophTobler The integration errors would not be visible in terms of diverging, but in terms of the convergence in EKF2 showing a constant error and the filter never converging fully to the right offset. |
Used for VIO for example
raw will be used as unfiltered
We are moving towards this interface for onboard companion communication: https://dev.px4.io/en/middleware/micrortps.html |
My apologies @ChristophTobler, I didn't mean to post that comment here... not sure how I managed to accomplish that. I'll delete it now so that it doesn't mislead future readers... Again, my apology. |
We currently need to stream the raw/unfiltered gyro/accel values on the Snapdragon via Mavlink for VIO estimation (probably also on other platforms). Since the raw values don't seem to be used (was even inconsistent) I changed to to be the unfiltered values. There can then be streamed via the
SCALED_IMU
Mavlink msg.Hopefully, I got all drivers...