-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Sensor Data Coordinate Convention #1154
Comments
Yes, I think that's what should happen. Sensor data is also in NED. Try it out and if that's not the case we will look at it. |
@sytelus Many thanks for getting back to me. After further investigating it seems to be in NED. Currently in AirSim its all -ve, so I assume this is correct. I've fed the imu data into the MadgwickAHRS filter, and it also has a NED coordinate system. However I have to negate all the accel data, hence the question. Thanks James |
At complete resting state, the IMU should report -9.81 m/s^2 though the AirSim 1.2 API for simGetGroundTruthKinematics() does not account for gravity and it needs to be manually added after converting to body frame. |
Hi, I am confused here,the body frame also uses NED convention right? where z points downward, so if the drone is resting upright, shouldn't the downward pointing z direction acceleration reads +9.81 instead of -9.81? namely line 55 of ImuSimple.hpp: why subtract ground_truth.environment->getState().gravity |
I've read in the documentation that the NED coordinate convention is used throughout AirSim.
Can someone confirm that the sensor data coming into e.g. SimpleFlight is also in NED convention.
If the drone is upright I can see an acceleration of -9.81 m/s^2
If the drone is rolled 90 degrees on its right, I should see -9.81 on the Y axis?
If the drone is pitched 90 degrees forward (on its nose) I should see -9.81 on the X axis?
Can anyone confirm that my assumptions are correct.
I would really love to be able to pick up the drone and spin it around in the simulator to test this.
Many thanks..
James
The text was updated successfully, but these errors were encountered: