Make transformation framework pluggable #346
Commits on Sep 6, 2018
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
Make FrameTransformer::transform() return a PoseStamped and rename Fr…
…ameTransformer::lastAvailableTransform()
-
-
-
-
-
-
-
-
-
Add TransformationPanel to change transformation plugin at runtime
Note: The panel is not yet functional and requires the implementation of a TransformationManager
-
-
-
Use the TransformationManager in the TransformationPanel
The TransformationManager is used to retrieve the available plugins and to set the transform after selecting it in the GUI
-
Replace the button group by a PropertyTreeWidget with radio buttons
Now the design fits better with the rest of the panels. Note: The QTreeView does not allow to place QWidgets into the column, thus the self implemented radio button group.
-
-
-
-
-
-
Rename FrameTransformerTF to TFFrameTransformer
Equivalent to naming scheme for Displays, Panels, etc.
-
-
Move transformation_manager.hpp to include folder (to be used in rviz…
…_default_plugins)
-
-
Draw checkbox instead of radio button in RadioButtonProperty
This prevents rendering issues on macOS, where the radio button would be cut off on the bottom.
-
-
-
-
Add PluginInformation struct to rviz_common::Factory
This allows a factory user to conveniently get all information for one plugin.
-
-
Omit "FrameManager" from TFFrameManager plugin name
This is equivalent to how e.g. Displays are named
-
-
-
Refactor factories to work with PluginInfo
This improves the interaction with factories since all attributes of a plugin are immediately available and do not have to be retrieved separately.
-
Change design: from a base common class (OnlyTfCompatibleBaseDisplay)…
… to a delegate object (TransformerHandlerDelegate)
-
Expose a method to check if the current transformer is the correct on…
…e, to be used in the Display::update() method
-
-
-
-
Use TransformerGuard in TFDisplay to make it handle changes of the Fr…
…ameTransformer type
-
Add config loading and saving to TransformationManager
Also improves error handling.
-
-
Harmonize comment style in frame_transformer.hpp and add TODO for com…
…mented out method
-
-
-
-
-
-
-
Fix pluginlib icon loading crash on missing plugin
This also fixes the segfault on startup when the transformer to load is not available
-
-
-
-
-
-
-
-
Change name of class InternalFrameTransformer to TransformationLibrar…
…yConnector and of relative getters
-
-
-
-
-
-
Fix points marker behavior on missing transform
The old routine did not return when transformation fails, which resulted in strange visual behavior on Windows
-
Refactor TransformerGuard so that it checks the type of the transform…
…er itself rather than of the connector -To accomplish this modification the new method getTransformer had to be added to rviz_common::FrameManager
-
-
-
-
-
-
-
-
Update comment in transformer_guard.hpp and insert TF as default tran…
…sformer in visual testing config
-
Pass clock to tf2::Buffer contructor
-Necessary after PR ros2#340
-
-
- use "::WeakPtr" convention - add missing visibility macro to TransformationManager - retain information about tf exceptions when re-throwing - fix documentation typo
-
Remove reset button from transformer panel
- superfluous for small amount of checkboxes - Code-Style: reorder functions in transformer panel
-
Signed-off-by: William Woodall <william@osrfoundation.org>
-
use ++ suffix in loop iterators (cppcheck)
Signed-off-by: William Woodall <william@osrfoundation.org>
-
suppress Ogre warnings when interpreting single header (autocomplete)
Signed-off-by: William Woodall <william@osrfoundation.org>
-
initialize member variable in constructor
Signed-off-by: William Woodall <william@osrfoundation.org>
Commits on Sep 7, 2018
-
Implement workaround for missing Qt clicked event
- Clicking on the transformers view, sometimes the clicked event is only sent to the underlying property, not the tree view - This results in weird behaviour of the gui - With this fix, it can only (rarely) happen that the click does nothing
-