-
Notifications
You must be signed in to change notification settings - Fork 70
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
IMU sensors - Include alljoints-inertials_wrapper/remapper in YRI xml files #640
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.
Hi @martinaxgloria
CI is failing.
Probably I'm missing some configuration files, I'll take a look into |
As expected, the CI was falling due to some missing configuration files. Moreover, I noticed that the dry-run checks only the XML that the YRI uses by default, that is the one under |
Ok, I think I did not get your question. |
Aligned with @traversaro t2t, we decided to expose IMU measurements only in the yri default xml for the iCubs, while for ergoCubSN001 and ergoCubSN002 in all |
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 heaps for the fix, @martinaxgloria 🚀
As pointed out by @traversaro here, the files that expose via multipleanalogsensorsremapper the orientation measurements of the available IMUs are now included in the XML used by the yarprobotinterface
This PR will complete #636