robotology / icub-tech-support

Virtual repository that provides support requests for individual robots
GNU General Public License v2.0
20 stars 1 forks source link

ergoCub 1.1 S/N:001 – Problems when loading configuration files for torque control parameters #1718

Closed DanielePucci closed 5 months ago

DanielePucci commented 5 months ago

Robot Name 🤖

ergoCub 1.1 S/N:001

Request/Failure description

Today, we started with @isorrentino to work to test the Yoga demo on the real robot after the preliminary results in https://github.com/ami-iit/element_qp-reactive-control/issues/70.

So, we started modifying the files for loading the parameters of torque control. See the commit https://github.com/ami-iit/robots-configuration/commit/f6fe83cf51cf5e385477fbe4fdc2bd043313191e

Unfortunately, these parameters seem not loaded in the motor gui

image

And we cannot at the moment test the torque control since adding them manually for all the joints is a very long process

Detailed context

This is the output of yarprobotinterface output_interface.txt

@Nicogene @sgiraz @maggia80

Additional context

No response

How does it affect you?

No response

github-actions[bot] commented 5 months ago

⚠️ Detected missing mandatory information

Hi @DanielePucci 👋🏻

Some of the following points need your attention.

You are required to:

Please, mark the points above as solved once done. [^1]: To get to know the allowed options for the dropdown fields, you may peruse the template.

sgiraz commented 5 months ago

Hi @DanielePucci,

The Parser seems to parse correctly the values set in config files:

...

(stictionUp 0 0 0) (stictionDown 0 0 0) (kff 1 1 1) (viscousPos 0 0 0) (viscousNeg 0 0 0) (coulombPos 0 0 0) (coulombNeg 0 0 0) (velocityThres 0.1 0.1 0.1) (filterType 0 0 0) (ktau 176 176 -176)"), ("OTHER_CONTROL_PARAMETERS" [group] = "(deadZone 0.0049 0.0049 0.0049)")]

...

I'm not sure, but as a first quick view to the info provided, it could be a (wrong) cast from float to int that trunkates the 0.1 to 0. By the way, we'll perform some tests in during our next sprint (starting from Monday 22).

cc @Nicogene

DanielePucci commented 5 months ago

Hi @sgiraz, thanks for the hint. We tried to set all parameters as integers, but it does not seem to solve the issue

image

Here's the log log_ergocub-torso_yarprobotinterface_18098.txt

sgiraz commented 5 months ago

Hi @DanielePucci,

I fixed a bug in icub-main that was preventing the parsing of torque parameters when the (⚠️ deprecated) kbemf parameter was missing. Here's the PR that addresses the issue:

cc @isorrentino @GiulioRomualdi @AntonioConsilvio

DanielePucci commented 5 months ago

Thanks a lot @sgiraz, hopefully tomorrow we are going to give it a try!

isorrentino commented 5 months ago

Today, @DanielePucci and I tested the PR. Now everything works fine. Thanks a lot.

sgiraz commented 5 months ago

Hi @isorrentino, @DanielePucci

Thank you for your precious feedback.

I proceed closing this issue! ✅

Cc @Nicogene @AntonioConsilvio