Wrong visual about PointStamped radius
Closed this issue · 2 comments
Julien-Livet commented
I observed a vocabulary mistake about PointStamped display : the field "Radius" currently matches to the diameter (with TF display).
I think it is more simple to rename the field than correct the code (div by 2), probably https://github.com/ros-visualization/rviz/blob/5531206e77d8092dbdb6a50a5b1a3c23fe5e181e/src/rviz/default_plugin/point_display.cpp or here https://github.com/ros-visualization/rviz/blob/5531206e77d8092dbdb6a50a5b1a3c23fe5e181e/src/rviz/default_plugin/point_visual.cpp.
Your environment
- OS Version: e.g. Xubuntu 20.04
- ROS Distro: [Noetic]
- RViz, Qt, OGRE, OpenGl version as printed by rviz:
[ INFO] [1635843067.996297233]: compiled against Qt version 5.12.8
[ INFO] [1635843067.996307080]: compiled against OGRE version 1.9.0 (Ghadamon)
[ INFO] [1635843068.006034471]: Forcing OpenGl version 0.
[ INFO] [1635843068.251274587]: Stereo is NOT SUPPORTED
[ INFO] [1635843068.251392943]: OpenGL device: llvmpipe (LLVM 12.0.0, 128 bits)
[ INFO] [1635843068.251541995]: OpenGl version: 3,1 (GLSL 1,4).```
rhaschke commented
Well spotted!