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

Ros2 minimal add kr16 #1

Open
wants to merge 7 commits into
base: foxy
Choose a base branch
from
Open

Ros2 minimal add kr16 #1

wants to merge 7 commits into from

Conversation

mamueluth
Copy link
Member

No description provided.

@destogl
Copy link
Member

destogl commented Feb 10, 2023

When we merge rolling, then please take care of this Manuel's KR 16 version

@muritane muritane force-pushed the ros2-minimal-add-kr16 branch from 9f9ba64 to 7e6b556 Compare March 27, 2023 18:48
@muritane muritane force-pushed the ros2-minimal-add-kr16 branch from 7e6b556 to ff042a7 Compare March 27, 2023 18:56
Copy link
Member

@destogl destogl left a 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?

Copy link
Member

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.

Copy link
Member

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?

Copy link
Member

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?

Copy link
Member

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?

Copy link
Member

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>
Copy link
Member

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>
Copy link
Member

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>
Copy link
Member

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"?>
Copy link
Member

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

Copy link
Member

Choose a reason for hiding this comment

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

Probably also obsolete

Copy link
Member

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?

Base automatically changed from ros2-minimal to foxy March 27, 2023 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants