AP_Scripting: add REPL applet #27311
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
IamPete1
reviewed
Jun 16, 2024
|
Yeah, it would be nice, but it looked a bit troublesome. I also want coroutines and |
tpwrules
force-pushed
the
lua-script-repl
branch
from
a77d088
to
5f4a501
Compare
tpwrules
force-pushed
the
lua-script-repl
branch
from
5f4a501
to
0aa1794
Compare
The HAL always returns nullptr so the function always returns, which is handled already by the default.
The only caller is now gone and nobody ever overrode it to do return more than nullptr.
tpwrules
force-pushed
the
lua-script-repl
branch
from
0aa1794
to
aa1bcce
Compare
|
I've been using this a while and am quite happy with it. Seems to work reliably and has been very useful for my recent scripting PRs. Added a few features and tweaked some things (including removing some vestigial code and saving a few bytes of flash). I think it would be cool for it to be merged for others to use. |
tpwrules
force-pushed
the
lua-script-repl
branch
from
aa1bcce
to
d4526c3
Compare
|
looks nice! |
IamPete1
reviewed
Aug 3, 2024
tpwrules
force-pushed
the
lua-script-repl
branch
from
d4526c3
to
58b2f20
Compare
Abstracts the MAVLink SERIAL_CONTROL message to be usable as a Lua port, in particular the DEV_SHELL device supported by QGroundControl's MAVLink Console.
Implements a reasonably comfortable REPL accessible over serial entirely as a loadable script. Also accessible over MAVLink using QGroundControl's MAVLink Console and the new mavport module.
tpwrules
force-pushed
the
lua-script-repl
branch
from
58b2f20
to
ab759dc
Compare
IamPete1
approved these changes
Aug 5, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Implements a reasonably comfortable REPL accessible over serial entirely as a loadable script. Also accessible using QGroundControl's MAVLink Console feature, if so configured.
Removes never-implemented
SERIAL_CONSOLE_DEV_SHELLhandling and saves a few dozen bytes on Cube Orange.Tested in SITL and on the bench with a Cube Orange. Was able to run a couple examples through it.