search

moveit / moveit_setup_assistant

THIS REPO HAS MOVED TO https://github.com/ros-planning/moveit
8 stars 20 forks source link

Enable / Disable Collision Pairs is Broken #42

Closed JeremyZoss closed 10 years ago

JeremyZoss commented 11 years ago

In the MoveIt Setup Assistant, manually changing the Self-Collision "disabled" states seems to have no effect.

If you clear an auto-generated checkbox (enabling the pair), then leave to a different tab (e.g. Virtual Joints), and return to the "Self Collisions" tab, the checkbox will return to the original (checked) state. Similarly, manually disabling a particular collision pair will return to the original (unchecked) state when you leave and return. These changes are also not saved to the SRDF file.

I think the issue is in default_collisions_widget.cpp. It looks like the manual enable/disable functionality is implemented in the "toggleCheckBox" method, but this method is not connected to any of the form objects. As far as I can see, there are no references to this method other than its original definition.

pirobot commented 10 years ago

This still seems to be broken. )-: I am using the latest Hydro Debian installation under Ubuntu 12.04.

hersh commented 10 years ago

I'm looking into this now...