search

cpaxton / costar_stack

Integrated ROS capabilities for planning, predicate inference, gripper control, and perception for use with the KUKA LBR IIWA and Universal Robots.
http://cpaxton.github.io/2016/04/15/costar/
Other
86 stars 25 forks source link

SmartMove UI does not update list of waypoints #35

Closed cpaxton closed 7 years ago

cpaxton commented 7 years ago

Because there's no "Refresh waypoints" button and it used to do that on creation of a new UI

cpaxton commented 7 years ago

Hey Felix, so we want to solve this by exposing a refresh() function from the different UI elements, right?

Where would that be called from? I'd like to get this fixed over the next couple days if we can.

fjonath1 commented 7 years ago

We could make it refresh everytime we click the action button on UI on def component_selected_callback(self,name):

Alternatively, we can also call the refresh function on wherever instructor view used to remake the plugin with self.all_generators[name] = self.plugins[name]['module']()

Although I'm not sure whether we actually need to refresh it wherever instructor view used to remake the plugin, but I think it will cause little to no harm to do that.

fjonath1 commented 7 years ago

This issue is fixed on commit e6f386ba9f7aa3224ee36adfac3f71f01b0b9a9b