search

icub-tech-iit / ergocub-software

Main collector of ergoCub specific SW
https://icub-tech-iit.github.io/ergocub-software/
BSD 3-Clause "New" or "Revised" License
13 stars 18 forks source link

couplingXCubHandMk5 – Refactor the param parsing for using yarpDeviceParamParserGenerator #223

Open Nicogene opened 7 months ago

Nicogene commented 7 months ago

Task description

yarpDeviceParamParserGenerator is a new YARP functionality that allows to automatically generate the parsing of the paramenter and documentation for device drivers:

Definition of Done

The parse of the parameter has been ported

Nicogene commented 6 months ago

I did the required changes in this PR:

Unfortunately, the merging is blocked because we need to contextually update of the controlboard gyp for parsing the lists using find instead of findGroup

cc @randaz81 cc @pattacini

randaz81 commented 6 months ago

Unfortunately, the merging is blocked because we need to contextually update of the controlboard gyp for parsing the lists using find instead of findGroup

I haven't understood if I should do something in this respect?

Nicogene commented 6 months ago

Unfortunately, the merging is blocked because we need to contextually update of the controlboard gyp for parsing the lists using find instead of findGroup

I haven't understood if I should do something in this respect?

@traversaro made already the changes in this PR:

But probably we need to change all the ini of the controlboard for adding the parenthesis?

traversaro commented 6 months ago

But probably we need to change all the ini of the controlboard for adding the parenthesis?

Yes, it would be ideal if we could wait for https://github.com/robotology/gz-sim-yarp-plugins/issues/138, I think we can do it in a few days and do a release of gz-sim-yarp-plugins after that.