4 ➡️ 5 (#154)

Signed-off-by: Louise Poubel <[email protected]>

CMakeLists.txt

@@ -75,7 +75,7 @@ endif()
 
 #--------------------------------------
 # Find ignition-msgs
-ign_find_package(ignition-msgs7 REQUIRED)
+ign_find_package(ignition-msgs7 REQUIRED VERSION 7.2)
 set(IGN_MSGS_VER ${ignition-msgs7_VERSION_MAJOR})
 
 #--------------------------------------
@@ -85,7 +85,7 @@ set(IGN_PLUGIN_VER ${ignition-plugin1_VERSION_MAJOR})
 
 #--------------------------------------
 # Find SDFormat
-ign_find_package(sdformat11 REQUIRED)
+ign_find_package(sdformat11 REQUIRED VERSION 11.3)
 set(SDF_VER ${sdformat11_VERSION_MAJOR})
 
 set(IGN_SENSORS_PLUGIN_PATH ${CMAKE_INSTALL_PREFIX}/${CMAKE_INSTALL_LIBDIR})

Changelog.md

@@ -23,6 +23,41 @@
 
 ### Ignition Sensors 4.X.X
 
+### Ignition Sensors 4.2.0 (2021-07-12)
+
+1. Add API for enabling / disabling IMU orientation
+    * [Pull request #142](https://github.com/ignitionrobotics/ign-sensors/pull/142)
+
+1. Init will now set the nextUpdateTime to zero
+    * [Pull request #137](https://github.com/ignitionrobotics/ign-sensors/pull/137)
+
+1. Remove clamping from lidar noise
+    * [Pull request #132](https://github.com/ignitionrobotics/ign-sensors/pull/132)
+
+1. Remove tools/code_check and update codecov
+    * [Pull request #130](https://github.com/ignitionrobotics/ign-sensors/pull/130)
+
+1. Disable macOS workflow
+    * [Pull request #124](https://github.com/ignitionrobotics/ign-sensors/pull/124)
+
+1. 👩‍🌾 Disable tests that consistently fail on macOS
+    * [Pull request #121](https://github.com/ignitionrobotics/ign-sensors/pull/121)
+
+1. Master branch updates
+    * [Pull request #106](https://github.com/ignitionrobotics/ign-sensors/pull/106)
+
+1. 👩‍🌾 Clear Windows warnings (backport #58)
+    * [Pull request #102](https://github.com/ignitionrobotics/ign-sensors/pull/102)
+
+1. Update thermal camera tutorial - include varying temp. objects
+    * [Pull request #79](https://github.com/ignitionrobotics/ign-sensors/pull/79)
+
+1. Fix macOS/windows tests that failed to load library
+    * [Pull request #60](https://github.com/ignitionrobotics/ign-sensors/pull/60)
+
+1. Removed issue & PR templates
+    * [Pull request #99](https://github.com/ignitionrobotics/ign-sensors/pull/99)
+
 ### Ignition Sensors 4.1.0 (2021-02-10)
 
 1. Added issue and PR templates.
@@ -66,6 +101,30 @@
 
 ## Ignition Sensors 3
 
+### Ignition Sensors 3.X.X (202X-XX-XX)
+
+### Ignition Sensors 3.3.0 (2021-08-26)
+
+1. 👩‍🌾 Print debug messages when sensors advertise topics
+    * [Pull request #151](https://github.com/ignitionrobotics/ign-sensors/pull/151)
+
+1. Publish performance sensor metrics.
+    * [Pull request #146](https://github.com/ignitionrobotics/ign-sensors/pull/146)
+
+1. CI and infrastructure
+    * [Pull request #130](https://github.com/ignitionrobotics/ign-sensors/pull/130)
+    * [Pull request #150](https://github.com/ignitionrobotics/ign-sensors/pull/150)
+    * [Pull request #106](https://github.com/ignitionrobotics/ign-sensors/pull/106)
+
+1. 👩‍🌾 Disable tests that consistently fail on macOS
+    * [Pull request #121](https://github.com/ignitionrobotics/ign-sensors/pull/121)
+
+1. 👩‍🌾 Clear Windows warnings (backport #58)
+    * [Pull request #58](https://github.com/ignitionrobotics/ign-sensors/pull/58)
+
+1. Fix macOS/windows tests that failed to load library (backport #60)
+    * [Pull request #60](https://github.com/ignitionrobotics/ign-sensors/pull/60)
+
 ### Ignition Sensors 3.2.0 (2021-02-08)
 
 1. Apply noise to lidar point cloud.

include/ignition/sensors/ImuSensor.hh

@@ -118,13 +118,27 @@ namespace ignition
       public: math::Quaterniond OrientationReference() const;
 
       /// \brief Get the orienation of the imu with respect to reference frame
-      /// \return Orientation in reference frame
+      /// \return Orientation in reference frame. If orientation is not
+      /// enabled, this will return the last computed orientation before
+      /// orientation is disabled or identity Quaternion if orientation has
+      /// never been enabled.
       public: math::Quaterniond Orientation() const;
 
       /// \brief Set the gravity vector
       /// \param[in] _gravity gravity vector in meters per second squared.
       public: void SetGravity(const math::Vector3d &_gravity);
 
+      /// \brief Set whether to output orientation. Not all imu's generate
+      /// orientation values as they use filters to produce orientation
+      /// estimates.
+      /// \param[in] _enabled True to publish orientation data, false to leave
+      /// the message field empty.
+      public: void SetOrientationEnabled(bool _enabled);
+
+      /// \brief Get whether or not orientation is enabled.
+      /// \return True if orientation is enabled, false otherwise.
+      public: bool OrientationEnabled() const;
+
       /// \brief Get the gravity vector
       /// \return Gravity vectory in meters per second squared.
       public: math::Vector3d Gravity() const;

include/ignition/sensors/Sensor.hh

@@ -27,6 +27,7 @@
 #pragma warning(pop)
 #endif
 
+#include <chrono>
 #include <memory>
 #include <string>
 
@@ -188,6 +189,14 @@ namespace ignition
       /// \return True if a valid topic was set.
       public: bool SetTopic(const std::string &_topic);
 
+      /// \brief Get flag state for enabling performance metrics publication.
+      /// \return True if performance metrics are enabled, false otherwise.
+      public: bool EnableMetrics() const;
+
+      /// \brief Set flag to enable publishing performance metrics
+      /// \param[in] _enableMetrics True to enable.
+      public: void SetEnableMetrics(bool _enableMetrics);
+
       /// \brief Get parent link of the sensor.
       /// \return Parent link of sensor.
       public: std::string Parent() const;
@@ -222,6 +231,12 @@ namespace ignition
       public: void AddSequence(ignition::msgs::Header *_msg,
                   const std::string &_seqKey = "default");
 
+      /// \brief Publishes information about the performance of the sensor.
+      ///        This method is called by Update().
+      /// \param[in] _now Current time.
+      public: void PublishMetrics(
+        const std::chrono::duration<double> &_now);
+
       IGN_COMMON_WARN_IGNORE__DLL_INTERFACE_MISSING
       /// \internal
       /// \brief Data pointer for private data

src/AirPressureSensor.cc

@@ -122,6 +122,9 @@ bool AirPressureSensor::Load(const sdf::Sensor &_sdf)
     return false;
   }
 
+  igndbg << "Air pressure for [" << this->Name() << "] advertised on ["
+         << this->Topic() << "]" << std::endl;
+
   // Load the noise parameters
   if (_sdf.AirPressureSensor()->PressureNoise().Type() != sdf::NoiseType::NONE)
   {

src/AltimeterSensor.cc

@@ -100,6 +100,9 @@ bool AltimeterSensor::Load(const sdf::Sensor &_sdf)
     return false;
   }
 
+  igndbg << "Altimeter data for [" << this->Name() << "] advertised on ["
+         << this->Topic() << "]" << std::endl;
+
   // Load the noise parameters
   if (_sdf.AltimeterSensor()->VerticalPositionNoise().Type()
       != sdf::NoiseType::NONE)

src/CameraSensor.cc

@@ -268,6 +268,9 @@ bool CameraSensor::Load(const sdf::Sensor &_sdf)
     return false;
   }
 
+  igndbg << "Camera images for [" << this->Name() << "] advertised on ["
+         << this->Topic() << "]" << std::endl;
+
   if (!this->AdvertiseInfo())
     return false;
 
@@ -507,16 +510,7 @@ bool CameraSensor::AdvertiseInfo()
   }
   this->dataPtr->infoTopic += "/camera_info";
 
-  this->dataPtr->infoPub =
-      this->dataPtr->node.Advertise<ignition::msgs::CameraInfo>(
-      this->dataPtr->infoTopic);
-  if (!this->dataPtr->infoPub)
-  {
-    ignerr << "Unable to create publisher on topic["
-      << this->dataPtr->infoTopic << "].\n";
-  }
-
-  return this->dataPtr->infoPub;
+  return this->AdvertiseInfo(this->dataPtr->infoTopic);
 }
 
 //////////////////////////////////////////////////
@@ -529,9 +523,14 @@ bool CameraSensor::AdvertiseInfo(const std::string &_topic)
       this->dataPtr->infoTopic);
   if (!this->dataPtr->infoPub)
   {
-    ignerr << "Unable to create publisher on topic["
+    ignerr << "Unable to create publisher on topic ["
       << this->dataPtr->infoTopic << "].\n";
   }
+  else
+  {
+    igndbg << "Camera info for [" << this->Name() << "] advertised on ["
+           << this->dataPtr->infoTopic << "]" << std::endl;
+  }
 
   return this->dataPtr->infoPub;
 }

src/DepthCameraSensor.cc

@@ -284,6 +284,9 @@ bool DepthCameraSensor::Load(const sdf::Sensor &_sdf)
     return false;
   }
 
+  igndbg << "Depth images for [" << this->Name() << "] advertised on ["
+         << this->Topic() << "]" << std::endl;
+
   if (!this->AdvertiseInfo())
     return false;
 
@@ -298,6 +301,9 @@ bool DepthCameraSensor::Load(const sdf::Sensor &_sdf)
     return false;
   }
 
+  igndbg << "Points for [" << this->Name() << "] advertised on ["
+         << this->Topic() << "/points]" << std::endl;
+
   // Initialize the point message.
   // \todo(anyone) The true value in the following function call forces
   // the xyz and rgb fields to be aligned to memory boundaries. This is need

src/GpuLidarSensor.cc

@@ -146,6 +146,9 @@ bool GpuLidarSensor::Load(const sdf::Sensor &_sdf)
     return false;
   }
 
+  igndbg << "Lidar points for [" << this->Name() << "] advertised on ["
+         << this->Topic() << "]" << std::endl;
+
   this->initialized = true;
 
   return true;