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.

Sign up for GitHub

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

Jump to bottom

Move entire baxterlgv7 #2622

Merged
merged 8 commits into from
Jan 24, 2018
Merged

Move entire baxterlgv7 #2622

merged 8 commits into from
Jan 24, 2018

Conversation

pazeshun
Copy link
Collaborator

@pazeshun pazeshun commented Jan 17, 2018 •
edited
Loading

This PR adds what I hadn't finished in #2616

  • Add baxterlgv7.launch having the same role as baxter.launch
  • Add moveit for baxterlgv7
  • Add sanity check code for baxterlgv7 grippers
  • Refactor configs and launches of moveit for both current baxter and baxterlgv7
  • (minor change) Fix robot name in xacro

Video

https://drive.google.com/open?id=17pm61w8mhTPbMPij-iadqmwhHluaxyAt
(Please download to see clearly)
I confirmed real robot also moved correctly when I took video

@pazeshun pazeshun self-assigned this Jan 17, 2018
@pazeshun pazeshun changed the title Move entire baxterlgv7 [WIP] Move entire baxterlgv7 Jan 17, 2018
@pazeshun pazeshun changed the title [WIP] Move entire baxterlgv7 Move entire baxterlgv7 Jan 17, 2018
@pazeshun
Copy link
Collaborator Author

I'll separate directory to distinguish configs for baxterlgv7 from configs for baxter in jsk_arc2017_baxter

@pazeshun pazeshun changed the title Move entire baxterlgv7 [WIP] Move entire baxterlgv7 Jan 18, 2018
@pazeshun pazeshun changed the title [WIP] Move entire baxterlgv7 Move entire baxterlgv7 Jan 18, 2018
@pazeshun pazeshun changed the title Move entire baxterlgv7 [WIP] Move entire baxterlgv7 Jan 18, 2018
@pazeshun pazeshun changed the title [WIP] Move entire baxterlgv7 Move entire baxterlgv7 Jan 18, 2018
@pazeshun
Copy link
Collaborator Author

pazeshun commented Jan 18, 2018 •
edited
Loading

Tested with both current baxter and baxterlgv7

@pazeshun
Copy link
Collaborator Author

Tested with real robots again

@pazeshun
Copy link
Collaborator Author

@knorth55 Can you review this again?

@pazeshun
Copy link
Collaborator Author

Please merge this

@knorth55 knorth55 merged commit 9636ecf into start-jsk:master Jan 24, 2018
@pazeshun pazeshun deleted the move-whole-baxterlgv7 branch January 24, 2018 08:12
@pazeshun
Copy link
Collaborator Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@knorth55 knorth55 knorth55 approved these changes

Assignees

@pazeshun pazeshun

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pazeshun @knorth55