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

Requirements for panda_moveit_config release #193

Open
10 of 13 tasks
rhaschke opened this issue Nov 7, 2021 · 3 comments
Open
10 of 13 tasks

Requirements for panda_moveit_config release #193

rhaschke opened this issue Nov 7, 2021 · 3 comments

Comments

@rhaschke
Copy link
Contributor

rhaschke commented Nov 7, 2021

The new release of https://github.com/ros-planning/panda_moveit_config is blocked by a few things in franka_ros:

I have merged these (and the first item from the nice-to-have list) into #188 for convenience.

Nice to have

Optional

Open issues

@rickstaa
Copy link
Contributor

rickstaa commented Nov 9, 2021

@rhaschke Thanks for collecting all our issues into an overview. One small suggestion can you change

GripperCommand doesn't signal convergence (but movements are correctly executed)

to

GripperCommand doesn't signal convergence (but movements are correctly executed) (see #173)

The problem is explained in detail in that pull request. Further, another related issue that should be mentioned is that #146, which was merged to the devel branch, broke the gripper actions (see #179).

@gollth
Copy link
Contributor

gollth commented Nov 10, 2021

For the record, @rhaschke shall we change PR #173 to Issue #172 on the checklist? (as discussed here)

@rickstaa
Copy link
Contributor

A few things block the new release of https://github.com/ros-planning/panda_moveit_configrankaemika/franka_ros/pull/191)

@rhaschke Just for tracking purposes. The motion generators have been implemented in 2514ae1.

gollth added a commit that referenced this issue Jun 14, 2022
…resholds-parameter to develop

* commit '2a14287d579571060d343ec76ea51d214a28ac62':
  FIX: Read force threshold from correct parameter
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

No branches or pull requests

3 participants