-
Notifications
You must be signed in to change notification settings - Fork 18.1k
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_Scripting: add send named float binding #13753
Conversation
29aea9a
to
93dc7ae
Compare
93dc7ae
to
467003a
Compare
rebased, I still think this is very useful and we don't have the direct MAVLink method yet |
That's actually on my to be rebased and merge within the next month list...
If I get it rebased can you test it?
…On Fri, Feb 12, 2021, 18:17 Peter Hall ***@***.***> wrote:
rebased, I still think this is very useful and we don't have the direct
MAVLink method yet
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#13753 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAEKTCADZWMJ4KYN52OWO5LS6XHKRANCNFSM4LDPO53A>
.
|
@WickedShell happy to test, I still think this method deserves its own binding, its very useful for debugging. |
I guess once the general MAVLink stuff is in I'd rather not, simply to try and minimize flash consumption, and reduce the general overhead we impose on every script. I can give a straight example of sending a named float though, which could be easily used in other scripts. (I know, obligatory include comments here) |
This allows scripts to send named floats, this is useful for streaming interesting numbers to the GCS
ArduPilot/MissionPlanner#2346