Skip to content

Feature/irb6660#113

Open
wlg1984 wants to merge 8 commits intoros-industrial:kinetic-develfrom
kinneyjustin:feature/irb6660
Open

Feature/irb6660#113
wlg1984 wants to merge 8 commits intoros-industrial:kinetic-develfrom
kinneyjustin:feature/irb6660

Conversation

@wlg1984
Copy link
Copy Markdown

@wlg1984 wlg1984 commented Mar 22, 2019

Adding irb6660 (and variants) support and moveit config

Copy link
Copy Markdown

@kinneyjustin kinneyjustin left a comment

Choose a reason for hiding this comment

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

looks good

kinneyjustin
kinneyjustin previously approved these changes May 8, 2019
Copy link
Copy Markdown
Contributor

@cjue cjue left a comment

Choose a reason for hiding this comment

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

From my view this PR still needs some changes.

@@ -1,4 +0,0 @@
cmake_minimum_required(VERSION 2.8.3)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These two abb_experimenal/ files should not be deleted.

<!--
Manipulator specific version of the state visualizer.

Defaults provided for irb_6650s_90:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

wrong robot model here and in the usage line

@@ -0,0 +1,11 @@
moveit_setup_assistant_config:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

As moveit configs are usually application dependant they should probably not be part of this pull request.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants