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

use transformable_markers for kiva_pod #2541

Merged
merged 1 commit into from
Aug 8, 2017

Conversation

knorth55
Copy link
Member

@knorth55 knorth55 commented Aug 7, 2017

for jsk_2016_01_baxter_apc

Copy link
Collaborator

@pazeshun pazeshun left a comment

Choose a reason for hiding this comment

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

Can you also fix baxterrgv5.launch?

@knorth55 knorth55 force-pushed the trasformable-kivapod-marker branch from f0ec1f8 to a650c7b Compare August 7, 2017 14:54
@knorth55
Copy link
Member Author

knorth55 commented Aug 7, 2017

updated

@@ -393,7 +393,7 @@
(setq box-list (send box-msg :boxes))
(setq bin-list (list :a :b :c :d :e :f :g :h :i :j :k :l))
(dolist (bin bin-list)
(setf (gethash bin _bin-boxes) (car box-list))
(sethash bin _bin-boxes- (car box-list))
Copy link
Member

Choose a reason for hiding this comment

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

must be _bin-boxes, right?

Copy link
Member

Choose a reason for hiding this comment

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

@pazeshun Could you please the naming of slots in baxter-rgv5.l?

Copy link
Collaborator

@pazeshun pazeshun Aug 8, 2017

Choose a reason for hiding this comment

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

Sorry for late response.
I'll change slots of baxterrgv5-interface.l and baxterrgv5.l after this PR.

@knorth55 knorth55 force-pushed the trasformable-kivapod-marker branch from a650c7b to 8d25a85 Compare August 8, 2017 03:39
@wkentaro wkentaro merged commit 238b1d5 into start-jsk:master Aug 8, 2017
@wkentaro wkentaro added this to the 4.1.0 milestone Aug 8, 2017
@knorth55 knorth55 deleted the trasformable-kivapod-marker branch August 9, 2017 05:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants