Renamed ignition_frame_id to gz_frame_id and fix warning (#296)

Signed-off-by: Alejandro Hernández Cordero <[email protected]>
gazebosim · Dec 6, 2022 · ea7d33a · ea7d33a
1 parent 8e8dfa8
commit ea7d33a
Show file tree

Hide file tree

Showing 6 changed files with 20 additions and 10 deletions.
diff --git a/src/Sensor.cc b/src/Sensor.cc
@@ -155,13 +155,23 @@ bool SensorPrivate::PopulateFromSDF(const sdf::Sensor &_sdf)
   sdf::ElementPtr element = _sdf.Element();
   if (element)
   {
-    if (element->HasElement("ignition_frame_id"))
+    if (element->HasElement("gz_frame_id"))
     {
-      this->frame_id = element->Get<std::string>("ignition_frame_id");
+      this->frame_id = element->Get<std::string>("gz_frame_id");
     }
     else
     {
-      this->frame_id = this->name;
+      // TODO(ahcorde): Remove this deprecation in gz-sensors8
+      if (element->HasElement("ignition_frame_id"))
+      {
+        gzwarn << "The `ignition_frame_id` tag is deprecated. "
+               << "Please use `gz_frame_id` instead." << std::endl;
+        this->frame_id = element->Get<std::string>("ignition_frame_id");
+      }
+      else
+      {
+        this->frame_id = this->name;
+      }
     }
   }
 

diff --git a/test/integration/triggered_camera.cc b/test/integration/triggered_camera.cc
@@ -125,7 +125,7 @@ void TriggeredCameraTest::ImagesWithBuiltinSDF(const std::string &_renderEngine)
         "/test/integration/TriggeredCameraPlugin_imagesWithBuiltinSDF";
     WaitForMessageTestHelper<gz::msgs::Image> helper(imageTopic);
     mgr.RunOnce(std::chrono::steady_clock::duration::zero(), true);
-    EXPECT_TRUE(helper.WaitForMessage()) << helper;
+    EXPECT_TRUE(helper.WaitForMessage(10s)) << helper;
   }
 
   // test removing sensor

diff --git a/test/sdf/camera_intrinsics.sdf b/test/sdf/camera_intrinsics.sdf
@@ -4,7 +4,7 @@
     <link name="link1">
       <sensor name="camera_without_intrinsics_tag" type="camera">
         <update_rate>10</update_rate>
-        <ignition_frame_id>base_camera</ignition_frame_id>
+        <gz_frame_id>base_camera</gz_frame_id>
         <topic>/camera1/image</topic>
         <camera>
           <horizontal_fov>1.05</horizontal_fov>
@@ -17,7 +17,7 @@
       </sensor>
       <sensor name="camera_with_intrinsics_tag" type="camera">
         <update_rate>10</update_rate>
-        <ignition_frame_id>base_camera</ignition_frame_id>
+        <gz_frame_id>base_camera</gz_frame_id>
         <topic>/camera2/image</topic>
         <camera>
           <horizontal_fov>1.05</horizontal_fov>

diff --git a/test/sdf/camera_sensor_builtin.sdf b/test/sdf/camera_sensor_builtin.sdf
@@ -4,7 +4,7 @@
     <link name="link1">
       <sensor name="camera1" type="camera">
         <update_rate>10</update_rate>
-        <ignition_frame_id>base_camera</ignition_frame_id>
+        <gz_frame_id>base_camera</gz_frame_id>
         <topic>/test/integration/CameraPlugin_imagesWithBuiltinSDF</topic>
         <camera>
           <horizontal_fov>1.05</horizontal_fov>

diff --git a/test/sdf/camera_sensor_l16_builtin.sdf b/test/sdf/camera_sensor_l16_builtin.sdf
@@ -4,7 +4,7 @@
     <link name="link1">
       <sensor name="camera1" type="camera">
         <update_rate>10</update_rate>
-        <ignition_frame_id>base_camera</ignition_frame_id>
+        <gz_frame_id>base_camera</gz_frame_id>
         <topic>/images_l16</topic>
         <camera>
           <horizontal_fov>1.05</horizontal_fov>

diff --git a/test/sdf/camera_sensor_l8_l16_builtin.sdf b/test/sdf/camera_sensor_l8_l16_builtin.sdf
@@ -4,7 +4,7 @@
     <link name="link1">
       <sensor name="camera_8_bit" type="camera">
         <update_rate>10</update_rate>
-        <ignition_frame_id>base_camera</ignition_frame_id>
+        <gz_frame_id>base_camera</gz_frame_id>
         <topic>/images_l8</topic>
         <camera>
           <horizontal_fov>1.05</horizontal_fov>
@@ -26,7 +26,7 @@
       </sensor>
       <sensor name="camera_16_bit" type="camera">
         <update_rate>10</update_rate>
-        <ignition_frame_id>base_camera</ignition_frame_id>
+        <gz_frame_id>base_camera</gz_frame_id>
         <topic>/images_l16</topic>
         <camera>
           <horizontal_fov>1.05</horizontal_fov>