Skip to content
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

Improve serial device for Dextra hand #216

Merged
merged 20 commits into from
Jul 12, 2019
Merged

Improve serial device for Dextra hand #216

merged 20 commits into from
Jul 12, 2019

Conversation

PeterBowman
Copy link
Member

Closes #176. Implements the controlboardwrapper2 subdevice that interfaces with the Teensy via USB serial port.

  • the YARP device has been renamed from DextraControlboardUSB to DextraSerialControlboard
  • instantiates YARP's serialport device under the hood to take care of comms
    • pros: no need to meddle with low-level details, multiplatform (no longer requires a UNIX system)
    • cons: requires ACE (recompile YARP with -DSKIP_ACE:BOOL=OFF)
  • Teensy-compatible protocol ensured via C++ port of synapse.py (see also firmware side)
  • unused interface implementations have been removed: torque control, velocity control (this one is required by the YARP wrapper, therefore I left it, but all methods return false), interaction mode
  • I'd rather nuke position control as well and move motion commands to the position direct implementation, leaving a few methods such as positionMove solely for ease of use via terminal+RPC; currently starts at position control mode, though, since yarpmotorgui does not let me enable the desired position sliders in position direct mode (future work)
  • implements IAxisInfo, which labels joints accordingly (see capture below)
  • yes, now you can connect to the RPC port and set pos any Dextra's joint as expected, even via yarpmotorgui (can be started with all robot parts + both dextras, thus allowing to move robot + hands within the same GUI)

dextra-yarpmotorgui

@coveralls
Copy link

coveralls commented Jul 5, 2019

Coverage Status

Coverage remained the same at ?% when pulling 14a9c8d on fix-176-dextra into fc417a5 on develop.

Copy link
Member

@jgvictores jgvictores left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great! Thanks a lot!!!

@jgvictores jgvictores merged commit 14ba9f5 into develop Jul 12, 2019
@jgvictores jgvictores deleted the fix-176-dextra branch July 12, 2019 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Develop serial Dextra device
3 participants