-
Notifications
You must be signed in to change notification settings - Fork 102
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
Iris: update iris_with_gimbal model to 3d gimbal #102
Iris: update iris_with_gimbal model to 3d gimbal #102
Conversation
- Match gimbal channels to servo gimbal wiki. - Reduce gains on controllers. - Change direction of pitch and yaw. Signed-off-by: Rhys Mainwaring <[email protected]>
Thanks so much for this. This is quite important for the high-altitude-non-GPS GSoC project! |
Hi @srmainwaring, What should MNT1_TYPE be set to? I think it will be either 1 (Servo) or 7 (BrushlessPWM?) |
Forgot to add that: yes MNT1_TYPE should be 1 for servo. Updated PR notes at top. |
Hi @srmainwaring, Sanket, Tridge and I were playing around with this today and noticed a couple of issues:
Anyway, thanks again for this though, it is on the critical path for both the HighAlt project and the Visual follow-me project! |
There's a couple of possible causes for the lag which I'll look into. The most likely is the PID gains on the servo joint controllers in the simulation.
I'll adjust the joint limits and update the AP params to match the specs for the SIYI A8
Finally, the iris model could also do with a tune. IIRC from some other analysis the default params for gazebo-iris are quite close to the limit where roll oscillations become noticeable. There could be some cross-coupling with the gimbal movement that is also contributing to stability. |
Hi @srmainwaring, Thanks very much! By the way, in the instructions at the top of the PR (and maybe eventually on the wiki or in the README) we should specify what these MNT1_xxx parameters should be set to:
and I wonder if we also need to set these values? They seem to be ok that the defaults 1100 ~ 1900 but just in case
|
Signed-off-by: Rhys Mainwaring <[email protected]>
Signed-off-by: Rhys Mainwaring <[email protected]>
Signed-off-by: Rhys Mainwaring <[email protected]>
@rmackay9 and @snktshrma - should be much snappier now. The limits now match the SIYI A8. Main change to joint control is reducing the damping (equivalent to increasing the PID gains and max force exerted). The original physics settings were carried over from the model from Gazebo Classic - and correspond to a very stiff joint, not that appropriate for a small gimbal. Figure: gimbal tracking ROI located at home position (location of axis), response is faster with new settings. |
cfdab56
to
9363aed
Compare
Signed-off-by: Rhys Mainwaring <[email protected]>
9363aed
to
5c1bf93
Compare
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.
Thanks again @srmainwaring for this!
@snktshrma has told me that the latest changes greatly improve the attitude control of the gimbal which resolves the biggest blocker he was facing in the high-altitude-non-gps project
Updates #86 which was on a
main
branch preventing maintainer edits.iris_with_gimbal
model as default in example worlds.Copter params
Figure: gimbal tracking ROI located at the home position while Iris circles
The
GstCameraPlugin
is also enabled by default. To view the video in QGC editApplication Settings > Video Settings
. SelectUDP h.264 Video Stream
and use the default port5600
.Testing
Tested on macOS Sonoma 14.5 and Ubuntu 22.04 (VM). Both running ArduPilot
master
with Gazebo Harmonic.