Skip to content

Commit

Permalink
Migrate InteractiveMarkerDisplay (#457)
Browse files Browse the repository at this point in the history
* Move interactive markers source files

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Get interactive markers to compile without linter errors

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Update plugin description

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Publisher moved into interactive marker client

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Fix conversion warnings

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Add special property for interactive marker namespace selection

Added some simple tests for the new property.

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Refactoring

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Make tf2::TransformException a base class of FrameTransformerException

This follows the tf2::BufferCoreInterface API, avoiding issues where third-party
libraries are expecting to deal with tf2 exceptions rather than rviz specific
exceptions (e.g. like interactive_markers).

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Only reset parent display when fixed frame changes

The full communication reset isn't necessary.

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Remove KEEP_ALIVE timer

Not necessary in ROS 2 port of interactive_markers.

Signed-off-by: Jacob Perron <jacob@openrobotics.org>
  • Loading branch information
jacobperron committed Sep 10, 2019
1 parent af50f17 commit c53e307
Show file tree
Hide file tree
Showing 24 changed files with 3,980 additions and 3,489 deletions.
389 changes: 0 additions & 389 deletions rviz/src/rviz/default_plugin/interactive_marker_display.cpp

This file was deleted.

Loading

0 comments on commit c53e307

Please sign in to comment.