Alternative implementation using function overloading

Signed-off-by: Michael Carroll <[email protected]>
gazebosim · Nov 8, 2021 · 37eade4 · 37eade4
1 parent d553e8f
commit 37eade4
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 7 deletions.
diff --git a/src/gui/plugins/component_inspector/AirPressure.cc b/src/gui/plugins/component_inspector/AirPressure.cc
@@ -64,6 +64,7 @@ Q_INVOKABLE void AirPressure::OnAirPressureNoise(
     double _stdDevBias, double _dynamicBiasStdDev,
     double _dynamicBiasCorrelationTime)
 {
+  igndbg << "here" << std::endl;
   ignition::gazebo::UpdateCallback cb =
       [=](EntityComponentManager &_ecm)
   {

diff --git a/src/gui/plugins/component_inspector/AirPressure.qml b/src/gui/plugins/component_inspector/AirPressure.qml
@@ -124,10 +124,10 @@ Rectangle {
           dynamicBiasStdDev: model.data[5]
           dynamicBiasCorrelationTime: model.data[6]
 
-          // Connect to the onNoiseUpdate signal in Noise.qml
-          Component.onCompleted: {
-            pressureNoise.onNoiseUpdate.connect(
-                componentInspector.onAirPressureNoise)
+          function onNoiseUpdate(_mean, _meanBias, _stdDev,
+              _stdDevBias, _dynamicBiasStdDev, _dynamicBiasCorrelationTime) {
+            componentInspector.onAirPressureNoise(_mean, _meanBias, _stdDev,
+                _stdDevBias, _dynamicBiasStdDev, _dynamicBiasCorrelationTime);
           }
         }
       }

diff --git a/src/gui/plugins/component_inspector/Noise.qml b/src/gui/plugins/component_inspector/Noise.qml
@@ -51,9 +51,9 @@ Rectangle {
 
   // Signal that a user of this component can connect to in order to receive
   // noise updates 
-  signal onNoiseUpdate(double _mean, double _meanBias, double _stdDev,
-      double _stdDevBias, double _dynamicBiasStdDev,
-      double _dynamicBiasCorrelationTime)
+  function onNoiseUpdate(_mean, _meanBias, _stdDev,
+      _stdDevBias, _dynamicBiasStdDev, _dynamicBiasCorrelationTime) {}
+
 
   // Display the main content
   Column {