-
Notifications
You must be signed in to change notification settings - Fork 69
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
ergoCubSN001 - neck files updated #564
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few points need attention @davidetome.
ergoCubSN001/hardware/motorControl/head-eb20-j0_1-mc_service.xml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @davidetome 👍🏻
Awaiting the CI before merging.
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. |
Awaiting the following PR to be merged before putting this in ready for review: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @davidetome, the PR is ok!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK
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. |
This PR updates XML files for the neck following tests on the setup
cc @pattacini @sgiraz