-
Notifications
You must be signed in to change notification settings - Fork 9
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
Ros2 minimal add kr16 #1
base: foxy
Are you sure you want to change the base?
Conversation
When we merge rolling, then please take care of this Manuel's KR 16 version |
a6cfaca
to
726d0a4
Compare
9f9ba64
to
7e6b556
Compare
7e6b556
to
ff042a7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@muritane thanks for rebasing this. Now we see that probably most of this PR is already added in other PRs. What do you think?
kuka_kr16_support/CMakeLists.txt
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file should be reverted to the version as it was. Except the roslauch_test
. But the test we will probably have to in another PR. We need to launch tests as they are done in UR repository.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this needed, or functionality is already there, but configuration in another file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't this file also already in the repository?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mean from the functionality perspective?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not all should be deleted here
<test_depend>roslaunch</test_depend> | ||
|
||
<exec_depend>industrial_robot_client</exec_depend> | ||
<exec_depend>joint_state_publisher</exec_depend> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
keep this
<exec_depend>kuka_resources</exec_depend> | ||
<exec_depend>robot_state_publisher</exec_depend> | ||
<exec_depend>rviz</exec_depend> | ||
<exec_depend>xacro</exec_depend> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
keep this
<exec_depend>kuka_resources</exec_depend> | ||
<exec_depend>robot_state_publisher</exec_depend> | ||
<exec_depend>rviz</exec_depend> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
keep this but as rviz2
@@ -1,13 +1,6 @@ | |||
<?xml version="1.0"?> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should also be replaced with the PR #12
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably also obsolete
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't this too added?
No description provided.