-
-
Notifications
You must be signed in to change notification settings - Fork 163
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
Why define gyro's data as acceleration and gps's second part as altitude #297
Comments
In ENU, the third value is altitude.
This is inherited from https://github.com/cyberbotics/webots/blob/master/projects/robots/dji/mavic/controllers/mavic2pro/mavic2pro.c, we should probably fix it. |
@lukicdarkoo , I suggested to add webots simulation on PX4/PX4-Autopilot#18221. Do you have any advice ? |
That would be lovely, but I am not sure whether it's our priority right now. Could you please create an issue in the BTW, there is already an ArduPilot integration: |
Does PX4 depend on ROS? I see some launch files there, but I thought PX4 is standalone. |
PX4 is standalone. |
Hi, is there any progress on PX4 SITL in Webots? I cannot see the issue has been created. |
https://github.com/cyberbotics/webots_ros2/blob/f5f33a9035bf3a1152a9f9df4e1001d5db3375dd/webots_ros2_mavic/webots_ros2_mavic/mavic_driver.py #L82
and
https://github.com/cyberbotics/webots/blob/5c3c33470190fad062591e6f681c7967982c070f/projects/robots/dji/mavic/controllers/mavic2pro/mavic2pro.c #L116
define gyro's data as acceleration, but actually velocity, as descriped at https://www.cyberbotics.com/doc/reference/gyro?tab-language=c#wb_gyro_get_values
https://github.com/cyberbotics/webots/blob/5c3c33470190fad062591e6f681c7967982c070f/projects/robots/dji/mavic/controllers/mavic2pro/mavic2pro.c #L115
define gps's second part as altitude, but longitude, as descriped at https://www.cyberbotics.com/doc/reference/gps?tab-language=c#wb_gps_get_values
The text was updated successfully, but these errors were encountered: