search

robotology / yarp-devices-forcetorque

YARP Drivers for various commercial Force Torque sensors.
4 stars 4 forks source link

Update yarprobotinterface_amti.xml #13

Closed isorrentino closed 4 years ago

isorrentino commented 5 years ago

Update ids platforms

traversaro commented 5 years ago

Why are we changing the order of the ids?

isorrentino commented 5 years ago

Check this comment. Probably the cables connected to the platforms have been inverted, because on Nexus Vicon the platform 1 and 2 did not match the real ones.

cc @claudia-lat

traversaro commented 5 years ago

Ack.

traversaro commented 4 years ago

Can we merge this change @fjandrad ?

fjandrad commented 4 years ago

@isorrentino Did we do some tests on this just to verify the ids? If yes then we can merge.

isorrentino commented 4 years ago

@fjandrad I did tests when I opened the PR. Then, I did not test it anymore.

claudia-lat commented 4 years ago

Since @isorrentino did the test, can we merge this PR @fjandrad?

traversaro commented 4 years ago

I guess we can merge this, if necessary we can always revert this PR.