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

PX4 Cntrl changes #33

Closed
EhrazImam opened this issue Nov 18, 2022 · 1 comment
Closed

PX4 Cntrl changes #33

EhrazImam opened this issue Nov 18, 2022 · 1 comment

Comments

@EhrazImam
Copy link

Hi, I hope you are doing well as i can see here @

nh.subscribe<sensor_msgs::Imu>("/mavros/imu/data", // Note: do NOT change it to /mavros/imu/data_raw !!!
why we cant use /mavros/imu/data_raw. As what i know difference between these two is one gives you orientation and one doesn't as we have already switched off the magnetometer why we are using it.
Thank you

@PanNeng-ZJU
Copy link
Collaborator

The orientation of imu is used in the 51 line of controller.cpp:
u.q = imu.q * odom.q.inverse() * q;

This was referenced Feb 23, 2023
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