gz-rviz
gz-rviz copied to clipboard
Remove tf subscription from frame manager and use tf buffer instead
Summary:
- Removes subscription of tf since buffer already does that internally
- 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.
- Remove locks from functions that are mutually exclusive
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.