-
Notifications
You must be signed in to change notification settings - Fork 104
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
Fix frame name for multipleFT sensors #821
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 @triccyx
As discussed, let's extend the fix to the single FT device too.
I will create a new PR only for the old FT sensor devices. |
Just giving @randaz81 a couple of days for reviewing the PR. |
I have the following concern about this multipleFT device (please note that I am not familiar with this device, and I don't know on which robot is currently employed).
|
@randaz81 the result XML section for multiple FT is something like this <group name="SENSORS">
<param name="id"> l_foot_ft1 l_foot_ft2 l_foot_ft3 </param>
<param name="board"> strain2 strain2 strain2 </param>
<param name="location"> CAN2:13 CAN1:13 CAN2:11 </param>
<param name="framename"> myframename1 myframename2 myframename3 </param>
</group> Should it be ok? Each sensor has its own frameName. |
Fix frame name for multipleFTSensors
This is the first step done for MultipleFtSensor to understand if this fix is correct @randaz81