Skip to content
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

couplingXCubHandMk5 – Refactor the param parsing for using yarpDeviceParamParserGenerator #223

Open
Nicogene opened this issue Feb 23, 2024 · 4 comments · May be fixed by #231
Open

couplingXCubHandMk5 – Refactor the param parsing for using yarpDeviceParamParserGenerator #223

Nicogene opened this issue Feb 23, 2024 · 4 comments · May be fixed by #231
Assignees
Labels
domain-software Related to Software prj-ergocub Related to ErgoCub Project team-fix Related to Team Fix

Comments

@Nicogene
Copy link
Member

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 Nicogene added domain-software Related to Software prj-ergocub Related to ErgoCub Project team-fix Related to Team Fix labels Feb 23, 2024
@Nicogene Nicogene self-assigned this Mar 7, 2024
@Nicogene
Copy link
Member Author

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
Copy link

randaz81 commented Apr 2, 2024

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
Copy link
Member Author

Nicogene commented Apr 4, 2024

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
Copy link
Member

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 robotology/gz-sim-yarp-plugins#138, I think we can do it in a few days and do a release of gz-sim-yarp-plugins after that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain-software Related to Software prj-ergocub Related to ErgoCub Project team-fix Related to Team Fix
Projects
None yet
3 participants