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

Removed panda_moveit_config dependency in franka_ros #132

Merged
merged 2 commits into from
Mar 18, 2021

Conversation

c-rizz
Copy link

@c-rizz c-rizz commented Mar 17, 2021

Hello, I have been installing franka_ros with ros noetic on ubuntu 20.04 and I noticed that the ros-noetic-franka-ros is not available on apt. By taking a quick look at jenkins (https://build.ros.org/job/Ndev__franka_ros__ubuntu_focal_amd64/4/console) it looks like the build is failing due to the panda_moveit_config dependency in franka_ros. This because panda_moveit_config is also not available in noetic. From what I understand this dependency is a remnant from when panda_moveit_config was a part of this repository, so it should be possible to safely remove it.

@fwalch
Copy link
Member

fwalch commented Mar 18, 2021

Hi,

it probably makes sense to remove the dependency, but that will not fix the build issues because franka_example_controllers also depends on panda_moveit_config. Related: moveit/panda_moveit_config#77

Probably we could remove franka_example_controllers from franka_ros as well, as its only an example package.. can you add this to your PR?

Thanks!

@c-rizz
Copy link
Author

c-rizz commented Mar 18, 2021

Done, just removed the dependency in package.xml

@fwalch fwalch merged commit 55cc935 into frankaemika:noetic-devel Mar 18, 2021
@fwalch
Copy link
Member

fwalch commented Mar 18, 2021

Thanks!

gollth added a commit that referenced this pull request Aug 3, 2021
…azebo

* commit '4adb450f218c576631bf5e52cc51941d57769de0':
  ADD: Implementation for Body Jacobian in `ModelKDL`
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.

None yet

2 participants