-
Notifications
You must be signed in to change notification settings - Fork 37
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
[MCControler] Update main robot name #423
[MCControler] Update main robot name #423
Conversation
Hi @antodld If I understand correctly, what you are trying to do is:
So for example: MainRobot: MyJVRC
robots:
MyJVRC:
module: JVRC1 would load This poses at least one problem, say you have a controller that supports multiple robots and you wish to rename a few, so you would have something like: MainRobot: MyJVRC # sometimes you switch to MyHRP4CR
robots:
MyJVRC:
module: JVRC1
MyHRP4CR:
module: HRP4CR with this configuration and your patch, both modules are always loaded I would suggest to write a patch that allows this syntax: MainRobot:
name: MyName
module: MyModule To solve this I suggest to add a |
Hello @gergondet The behavior described in your multi-robot example is the desired one. With this PR, whatever the main robot you chose in your |
Just to clarify, that's two robots at the same time at all times right? With the syntax I proposed you can get the same effect: MainRobot:
name: MyJVRC
module: JVRC1
robots:
MyJVRC:
module: JVRC1
MyHRP4CR
module: HRP4CR Would get you the same robots as: MainRobot:
name: MyHRP4CR
module: HRP4CR
robots:
MyJVRC:
module: JVRC1
MyHRP4CR
module: HRP4CR |
…l and its name exist in a controller config file, it will be added as a mainrobot
fe724f3
to
4cd496f
Compare
…cified config file
Added -- - [mc_control] Support for renaming the main robot via the configuration (#423) - [mc_rbdyn] Add `addBodySensor` to `mc_rbdyn::Robot` (#419) - [mc_rtc] Add support for any fixed size vector to Schema (#422) - [mc_rtc] Add `std::map<std::string, T>` support in Schema (#412) - [mc_solver] Add support for inactive joints in a collision (#416) Fixes -- - [StabilizerTask] Fix disabling when the robot is in the air (#411)
This PR allows to rename the main robot declared in
mc_rtc.yaml
as the one named in a controller configuration inrobots