From 46eb87018ad4fa4139edf477261d1c5232ac64a6 Mon Sep 17 00:00:00 2001 From: Ian Chen Date: Tue, 2 Feb 2021 16:11:43 -0800 Subject: [PATCH] add image suffix to thermal camera topic name Signed-off-by: Ian Chen --- Migration.md | 7 +++++++ src/rendering/RenderUtil.cc | 2 +- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/Migration.md b/Migration.md index d0406d0db3..3015f66f93 100644 --- a/Migration.md +++ b/Migration.md @@ -11,6 +11,13 @@ release will remove the deprecated code. `const std::vector &` instead of forcing a copy. The calling code should create a copy if it needs to modify the vector in some way. +* Default generated topic name for thermal cameras now includes the `/image` + suffix. The `camera_info` topic has also been fixed to include the sensor + name in the generated topic string. The naming scheme should be consistent + with a normal camera sensor. Topic changes: + * `//` -> `///image` + * `//camera_info` -> `///camera_info` + ## Ignition Gazebo 4.0.0 to 4.X.X * Ignition Gazebo 4.0.0 enabled double sided material by default but this diff --git a/src/rendering/RenderUtil.cc b/src/rendering/RenderUtil.cc index 132bf06c48..8ce6b1d798 100644 --- a/src/rendering/RenderUtil.cc +++ b/src/rendering/RenderUtil.cc @@ -645,7 +645,7 @@ void RenderUtilPrivate::CreateRenderingEntities( const std::string cameraSuffix{"/image"}; const std::string depthCameraSuffix{"/depth_image"}; const std::string rgbdCameraSuffix{""}; - const std::string thermalCameraSuffix{""}; + const std::string thermalCameraSuffix{"/image"}; const std::string gpuLidarSuffix{"/scan"}; // Treat all pre-existent entities as new at startup