Closed GiulioRomualdi closed 3 years ago
It should work if you comment out these lines,
Because these FTs were removed in the model, https://github.com/robotology/icub-models-generator/pull/186.
Also possibly remove the contact frames related to the upper leg in, https://github.com/robotology/whole-body-estimators/blob/8f6e78117ccc9f1895d592ab7aa66cb1d9700ae4/devices/wholeBodyDynamics/app/wholebodydynamics-icub-external-eight-fts-sim.xml#L9
WIth this patch, I was able to run wbd
diff --git a/devices/wholeBodyDynamics/app/launch-wholebodydynamics-icub-eight-fts-sim.xml b/devices/wholeBodyDynamics/app/launch-wholebodydynamics-icub-eight-fts-sim.xml
index b1abeeb..dc27c79 100644
--- a/devices/wholeBodyDynamics/app/launch-wholebodydynamics-icub-eight-fts-sim.xml
+++ b/devices/wholeBodyDynamics/app/launch-wholebodydynamics-icub-eight-fts-sim.xml
@@ -50,15 +50,15 @@
<param name="local"> /wholeBodyDynamics/r_arm_ft_sensor </param>
</device>
- <device name="left_upper_leg_strain" type="analogsensorclient">
- <param name="remote"> /icubSim/left_leg/analog:o </param>
- <param name="local"> /wholeBodyDynamics/l_leg_ft_sensor </param>
- </device>
-
- <device name="right_upper_leg_strain" type="analogsensorclient">
- <param name="remote"> /icubSim/right_leg/analog:o </param>
- <param name="local"> /wholeBodyDynamics/r_leg_ft_sensor </param>
- </device>
+ <!-- <device name="left_upper_leg_strain" type="analogsensorclient"> -->
+ <!-- <param name="remote"> /icubSim/left_leg/analog:o </param> -->
+ <!-- <param name="local"> /wholeBodyDynamics/l_leg_ft_sensor </param> -->
+ <!-- </device> -->
+
+ <!-- <device name="right_upper_leg_strain" type="analogsensorclient"> -->
+ <!-- <param name="remote"> /icubSim/right_leg/analog:o </param> -->
+ <!-- <param name="local"> /wholeBodyDynamics/r_leg_ft_sensor </param> -->
+ <!-- </device> -->
<device name="left_lower_leg_front_strain" type="analogsensorclient">
<param name="remote"> /icubSim/left_foot_front/analog:o </param>
diff --git a/devices/wholeBodyDynamics/app/wholebodydynamics-icub-external-eight-fts-sim.xml b/devices/wholeBodyDynamics/app/wholebodydynamics-icub-external-eight-fts-sim.xml
index d8efb10..925b035 100644
--- a/devices/wholeBodyDynamics/app/wholebodydynamics-icub-external-eight-fts-sim.xml
+++ b/devices/wholeBodyDynamics/app/wholebodydynamics-icub-external-eight-fts-sim.xml
@@ -27,10 +27,10 @@
<group name="WBD_OUTPUT_EXTERNAL_WRENCH_PORTS">
<param name="/wholeBodyDynamics/left_arm/endEffectorWrench:o">(l_hand,l_hand)</param>
<param name="/wholeBodyDynamics/right_arm/endEffectorWrench:o">(r_hand,l_hand)</param>
- <param name="/wholeBodyDynamics/left_foot_front/cartesianEndEffectorWrench:o">(l_foot_front,l_foot_front,l_foot_front)</param>
- <param name="/wholeBodyDynamics/left_foot_rear/cartesianEndEffectorWrench:o">(l_foot_rear,l_foot_rear,l_foot_rear)</param>
- <param name="/wholeBodyDynamics/right_foot_front/cartesianEndEffectorWrench:o">(r_foot_front,r_foot_front,r_foot_front)</param>
- <param name="/wholeBodyDynamics/right_foot_rear/cartesianEndEffectorWrench:o">(r_foot_rear,r_foot_rear,r_foot_rear)</param>
+ <param name="/wholeBodyDynamics/left_foot_front/cartesianEndEffectorWrench:o">(l_foot_front,l_sole,l_sole)</param>
+ <param name="/wholeBodyDynamics/left_foot_rear/cartesianEndEffectorWrench:o">(l_foot_rear,l_sole,l_sole)</param>
+ <param name="/wholeBodyDynamics/right_foot_front/cartesianEndEffectorWrench:o">(r_foot_front,r_sole,r_sole)</param>
+ <param name="/wholeBodyDynamics/right_foot_rear/cartesianEndEffectorWrench:o">(r_foot_rear,r_sole,r_sole)</param>
</group>
@@ -48,8 +48,8 @@
<!-- ft -->
<elem name="l_arm_ft_sensor">left_upper_arm_strain</elem>
<elem name="r_arm_ft_sensor">right_upper_arm_strain</elem>
- <elem name="l_leg_ft_sensor">left_upper_leg_strain</elem>
- <elem name="r_leg_ft_sensor">right_upper_leg_strain</elem>
+ <!-- <elem name="l_leg_ft_sensor">left_upper_leg_strain</elem> -->
+ <!-- <elem name="r_leg_ft_sensor">right_upper_leg_strain</elem> -->
<elem name="l_foot_front_ft_sensor">left_lower_leg_front_strain</elem>
<elem name="l_foot_rear_ft_sensor">left_lower_leg_rear_strain</elem>
<elem name="r_foot_front_ft_sensor">right_lower_leg_front_strain</elem>
Maybe we can just rename the configuration files to be specific to iCub3 (instead of eight-FTs-sim) and open a PR, to handle this in a clean manner and be in sync with the current developments.
For this, we rename the files and change the associated CMakeLists option
We can also just keep the config files for wbd of iCubGenova09 in robots-configuration with all the rest of files.
Hi @prashanthr05, @GiulioRomualdi, @traversaro, I just had the same issue. Can we properly fix this with a PR?
Actually, I am working on it. I am going to open it
I'm trying to run wbd with the latest model of iCubGazeboV3 https://github.com/robotology/icub-models/commit/93f735875d10053d731b77d5b61df7606380597c however, I got the following error:
I think this is related to https://github.com/robotology/icub-models-generator/pull/186
@prashanthr05 @traversaro if you guide me in the resolution of the problem I can open a PR