gz-rviz icon indicating copy to clipboard operation
gz-rviz copied to clipboard

Remove tf subscription from frame manager and use tf buffer instead

Open shrijitsingh99 opened this issue 5 years ago • 1 comments

Summary:

  1. Removes subscription of tf since buffer already does that internally
  2. Since now there is no way to generate an event on an update of tf frames, therefore fetch the latest frames from the frame manager when and where it is needed and not rely on events to automatically update the frames.
  3. Remove locks from functions that are mutually exclusive

shrijitsingh99 avatar Oct 14 '20 01:10 shrijitsingh99

Thank you for adding back the FrameListChanged event. Can you revert the changes in the AxesDisplay plugin and GlobalOptions plugin to update Combo-box on receiving the event? Everything else looks good.

Sarath18 avatar Oct 26 '20 13:10 Sarath18