robotology / robots-configuration

Contains robots configuration files
BSD 3-Clause "New" or "Revised" License
14 stars 69 forks source link

ergoCubSN001 - neck files updated #564

Closed davidetome closed 1 year ago

davidetome commented 1 year ago

This PR updates XML files for the neck following tests on the setup

⚠️ This mod applies only to the pitch and roll of the neck, other joints of the head have to be calibrated as soon as available

cc @pattacini @sgiraz

pattacini commented 1 year ago

CI still fails:

WARNING] ServiceParser::convert(): P3P is not a legal string for eObrd_connector_t
[WARNING] ServiceParser::parse_port_conn(): CONN:P3P is not a legal string for a eObrd_connector_t
[WARNING] ServiceParser::parse_actuator_port(): CONN:P3P is not a legal string for a pwm connector port
[ERROR] ServiceParser::check_motion() PROPERTIES.JOINTMAPPING.actuator.port not valid for item 0
[ERROR] ServiceParser::parseService() gets same errors parsing SERVICE MC group
[ERROR] embObjMC  BOARD head-eb20-j0_1 (IP 10.0.1.20)  cannot parse service
[ERROR] BOARD head-eb20-j0_1 (IP 10.0.1.20)  Missing motion control parameters in config file
[ERROR] |yarp.dev.PolyDriver|head-eb20-j0_1-mc| Driver <embObjMotionControl> was found but could not open

This is due to the fact that we haven't integrated yet the new FW/parser. Thus, I'll keep this PR in draft and come back to it once the FW/parser is updated.

pattacini commented 1 year ago

Thus, I'll keep this PR in draft and come back to it once the FW/parser is updated.

Awaiting the following PR to be merged before putting this in ready for review:

pattacini commented 1 year ago

Required upstream PR merged ✅

I'm rebuilding the docker image used by the CI via https://github.com/robotology/robots-configuration/actions/runs/6198027673.

Once done, I'll put this PR in ready-for-review.