roboticslab-uc3m / yarp-devices

A place for YARP devices
https://robots.uc3m.es/yarp-devices/
9 stars 7 forks source link

New more simple code for lacquey-fetch adapted to the new firmware #179

Closed rsantos88 closed 6 years ago

rsantos88 commented 6 years ago

Only one byte to send the order of the hand movement. Hand values sended to mbed:

coveralls commented 6 years ago

Coverage Status

Coverage remained the same at ?% when pulling a101bc52e883dd4f84788212b6104dc28b0e6d4e on mbed-lacquey-fetch into 0b4aaafca4da7bee67bc3b8be26cb2dec7da5aab on develop.

jgvictores commented 6 years ago

There was a reason for sending more than 3 different values: being able to control the grip torque.

Obviating that fact (it has passive compliance anyway), some of the minimum requirements would be the following:

rsantos88 commented 6 years ago

In views of the legacy-PIC code, protocol will be:

0xA0 -> Mantener abriendo
0xC0 -> Mantener cerrando
0xF0 -> Parar (dejar de actuar, enable a 0)
rsantos88 commented 6 years ago

Now, the mbed firmware of lacqueyFetch is compatible with the code of the old hand PIC. Therefore, tt's not necessary to do this PR. See the changes here: https://os.mbed.com/users/raulsr988/code/LacqueyFetch/rev/8ebb11ba21be/ It works fine.

jgvictores commented 6 years ago

Okay, perfect! Thanks, closing!!