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: fix SiYi doesn't zoom continuously while manual zoom control #24639
Conversation
fedeac0
to
36d930a
Compare
I think CI shouldn't fail now. @rmackay9 Could you please review this again. |
dc0b1ee
to
724f03c
Compare
Updated on top of my changes as suggested on discord. |
ffd6ec6
to
7eb50a9
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.
LGTM thanks.
I've tested this quite a bit on my A8 and ZR10. The absolute zoom doesn't work on my ZR10 but it does apparently work on @khanasif786's ZR10.
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
385a7bf
to
306a92f
Compare
changed according to @peterbarker suggestion for using a structure. |
Co-authored-by: Randy Mackay <rmackay9@yahoo.com>
While sending a Manual zoom (rate) command to SiYi. zoom stops multiple time in between. This Pull fixes this by sending the same command multiple time at 20Hz. Tested on Zr10.
Support issue link here
I find out one more bug that is probably SiYI side. that is when we watch the zoom value on the SiYI FPV app the values are different than real zoom. eg: on 30x sometime it tell 20x, 22x, 27x etc.