-
Notifications
You must be signed in to change notification settings - Fork 201
Add support for external blob values in robotinterface #3066
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
Add support for external blob values in robotinterface #3066
Conversation
Windows builds are failing, I'm going to submit a fix soon. |
|
Fixed, there's nothing a double pointer can't do 😏. |
c7fba65
to
8795d43
Compare
Should I rebase this and change the base branch to master according to the assigned milestone (YARP 3.10.0)? |
Hello. Is there any action I should perform here to clarify the intention of the PR or perhaps enhance the unit tests? Would it be possible to consider it for the 3.11.x release train (instead of 3.12 or later)? |
fyi @randaz81 |
We freezed this PR because it seems overcomplicated and difficult to maintain in the future. |
It is possible to externally (i.e. outside the XML robot configuration) override any parameters marked as
extern-name
with compound (lists) or simple (strings, integers...) values. This patch adds support for blobs, with unit testing (also covering external lists for completeness).Use case: roboticslab-uc3m/yarp-devices#271 (comment).