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

[baxtereus] add ctype in angle-vector methods for moveit #730

Merged
merged 3 commits into from Nov 29, 2016

Conversation

Projects
None yet
2 participants
@knorth55
Copy link
Member

commented Nov 21, 2016

What changed

  • add ctype in angle-vector for moveit
  • add ctype in angle-vector-sequence for moveit
  • use &key instead of &rest in baxter angle-vector methods

@knorth55 knorth55 force-pushed the knorth55:add-ctype branch from 5fc54b6 to f90952c Nov 21, 2016

@knorth55 knorth55 force-pushed the knorth55:add-ctype branch 4 times, most recently from 0907c9a to 82b5ffe Nov 23, 2016

@knorth55 knorth55 force-pushed the knorth55:add-ctype branch 3 times, most recently from 28a3581 to d03e5e5 Nov 28, 2016

use &key instead of &rest in baxter angle-vector
use &key instead of &rest args in :angle-vector methods in baxtereus and
refine codes.
@knorth55

This comment has been minimized.

Copy link
Member Author

commented Nov 29, 2016

@k-okada can you review and merge this?

@k-okada k-okada merged commit a37055b into jsk-ros-pkg:master Nov 29, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@knorth55 knorth55 deleted the knorth55:add-ctype branch Nov 29, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.