-
Notifications
You must be signed in to change notification settings - Fork 18k
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
AP_Mount: In Servo backend, report attitude as demanded servo angles #27072
AP_Mount: In Servo backend, report attitude as demanded servo angles #27072
Conversation
Hi @nexton-winjeel, Thanks for this. You've tested it of course? I'm sure you have but I just need to ask to be sure. |
This looks like it's failing an autotest. It could of course be that there's an issue with the autotest itself but in any case we will need to fix it I think.
|
Yeah, testing now on our fork (which is slightly different, as it's currently based on v4.4). I pushed this now while we're still testing in case there's any feedback that we need to incorporate.
|
Equivalent to yaw_range_valid().
…has_tilt_control()
… not target angles
68df5d7
to
7a47bd1
Compare
I've fixed the Copter mount tests, but now a Sub test is failing:
Can't reproduce locally. Looks to be unrelated to the changes in this PR? |
Sadly, known flapping test. |
Merged, thanks! |
Currently, the attitude quaternion reported by the Servo backend is the target angles, not the current angles. This PR changes the reported attitude to be the currently demanded servo angles.
FYI: @rmackay9