Skip to content

Commit

Permalink
Migrate Wrench Display (#396)
Browse files Browse the repository at this point in the history
* Move wrench display

* Make it compile + fix linters

* Add visual test

* Add unit tests for wrench_visual

* Refactoring of wrench visual

- make visual update length when user updates it
  in the display panel

* Minor refactoring of wrench_stamped_display

* Add USE_MATH_DEFINES for Windows compatibility

* Update readme

* Improve const-correctness for local variables

* Fix short class description for WrenchVisual

* Fix WrenchVisual test for larger width than length

* Change display name to WrenchDisplay

- other displays don't include the "stamped" part in their name
  • Loading branch information
Martin-Idel authored and jacobperron committed Jun 19, 2019
1 parent 58f0cc1 commit 6877a13
Show file tree
Hide file tree
Showing 18 changed files with 1,054 additions and 461 deletions.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ For some displays, the [documentation is updated](docs/FEATURES.md).
| Robot Model |
| Temperature |
| TF |
| Wrench |


### Not yet ported
Expand All @@ -48,7 +49,6 @@ These features have not been ported to `ros2/rviz` yet.
| Interactive Marker |
| Oculus |
| Pose With Covariance |
| Wrench |

Other features:
- Filtering of Topic lists by topic type
Expand Down
176 changes: 0 additions & 176 deletions rviz/src/rviz/default_plugin/wrench_display.cpp

This file was deleted.

64 changes: 0 additions & 64 deletions rviz/src/rviz/default_plugin/wrench_display.h

This file was deleted.

139 changes: 0 additions & 139 deletions rviz/src/rviz/default_plugin/wrench_visual.cpp

This file was deleted.

Loading

0 comments on commit 6877a13

Please sign in to comment.