icub-models-generator icon indicating copy to clipboard operation
icub-models-generator copied to clipboard

*_upper_leg_back_contact frames

Open traversaro opened this issue 8 years ago • 11 comments

@gabrielenava defined and added to the simmechanics generated models the (r/l)_upper_leg_contact frames, that are convenient for experiments in which the robot is sitting on its legs. After a day, I suggest to renamed this frames to *_upper_leg_back_contact to avoid confusion.

These frames work fine, but they have been just committed to the robot models contained in yarp-wholebodyinterface , and I will list here the things that need to be addressed to have them properly integrated in the model generation pipeline.

  • [x] Document their location in http://wiki.icub.org/wiki/ICub_Model_naming_conventions#URDF_frames,
  • [x] Add them to Creo model committed in the hardware repository and regenerate the shrinkwraps for the upper_leg links and then regenerate the simmechanics xml,
  • [x] Commit the generated file in icub-model-generator and regenerated all the URDFs ,
  • [ ] ~~Copy the generated models in yarp-wholebodyinterface.~~

cc @DanielePucci

traversaro avatar Feb 17 '17 09:02 traversaro

I asked for Mechanical engineers support, and they are aware of this issue and our requirements.

gabrielenava avatar Aug 28 '17 10:08 gabrielenava

The new iCub model contains also the leg contact frames.

gabrielenava avatar Sep 07 '17 18:09 gabrielenava

Updated the documentation in the wiki. As discussed with @gabrielenava, probably yarp-wholebodyinterface is deprecated, so we can close the issue? Or do we have to copy the models in another location?

fiorisi avatar May 04 '18 09:05 fiorisi

related issue: https://github.com/robotology/robotology-superbuild/issues/14

gabrielenava avatar Jun 04 '18 09:06 gabrielenava

I noticed that this frames are not present in the iCubNancy01 model, differently from the version manually patched in https://github.com/robotology/yarp-wholebodyinterface/blob/master/app/robots/iCubNancy01/model.urdf . However, I do not know if anyone is using those frames, as the controllers in whole-body-controllers does not seem to support anymore iCubNancy01, at least judging from the configuration files.

traversaro avatar Aug 23 '18 11:08 traversaro

Yes, at the moment the torque controllers only support icubGazeboSim, iCubGazeboV2_5, and iCubGenova02/04. The idea was to port from WBIToolbox-Controllers only the bare essentials in order to keep an ordered and maintainable structure of the repo, and then to add other robots/features only if necessary. I think for the moment we can proceed without iCubNancy01 and if necessary we can add it later. Or should we ask to the Nancy lab?

gabrielenava avatar Aug 23 '18 11:08 gabrielenava

cc @DanielePucci

traversaro avatar Aug 23 '18 11:08 traversaro

If it takes too long time to generate a no-battery-pack model with these new frames, we may take the iCubGenova02/04 are remove some torso mass to generate the iCubNancy01 model, right?

DanielePucci avatar Aug 23 '18 15:08 DanielePucci

The Creo-generated model without backpack (such as the iCubDarmstadt01) already contains this models (but at the moment the iCubNancy01 robot is still generated with the dh parameters hardcoded in the code because it uses arv v1). However, the question was if we want to have iCubNancy01 configuration files for the controllers. If we don't, I think we can deal with this model problem when we will work in adding back the configuration files to the controllers.

traversaro avatar Aug 23 '18 15:08 traversaro

So probably we can close this issue?

fiorisi avatar Aug 26 '18 18:08 fiorisi

I prefer to solve the iCubNancy01 problem before closing this issue, see also https://github.com/robotology/yarp-wholebodyinterface/issues/89 .

traversaro avatar Sep 12 '18 08:09 traversaro