-
Notifications
You must be signed in to change notification settings - Fork 29
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
Copter: update pose and lift plugin #43
Conversation
- Changed behaviour in Gazebo Harmonic. Signed-off-by: Rhys Mainwaring <[email protected]>
- iris_with_standoffs pose changed in ArduPilot/ardupilot_gazebo#68 Signed-off-by: Rhys Mainwaring <[email protected]>
Signed-off-by: Rhys Mainwaring <[email protected]>
Signed-off-by: Rhys Mainwaring <[email protected]>
@Ryanf55 - now that have been merged, this needs to be merged to fix the gimbal offset. Most of the changes are formatting, the key one is ardupilot_gz/ardupilot_gz_description/models/iris_with_lidar/model.sdf Lines 286 to 287 in 5dd3272
If you can approve please, I'll merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm other than one value. If it's right, just merge it.
@@ -69,13 +69,13 @@ | |||
|
|||
<include> | |||
<uri>model://runway</uri> | |||
<pose>0 0 0 0 0 1.57079632</pose> | |||
<pose degrees="true">-29 545 0 0 0 363</pose> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not 360 degrees?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not 360 degrees?
CUAV airport does not quite lie N-S and I've attempted to align with that as it's the default location for AP SITL. But 003 deg would have been better! I'll get those next time...
Update pose following:
Ensure
<cma>
element is set to zero in LiftDrag plugin following:Other changes