Only initialize empty series once fixes mem leak.

Author: john-michaelburke

resources/TrackingTabComponents/TrackingSignalsTab.qml

@@ -15,6 +15,7 @@ Item {
     property alias check_labels: trackingSignalsPoints.check_labels
     property alias num_labels: trackingSignalsPoints.num_labels
     property variant check_visibility: []
+    property var emptySeries: null
 
     TrackingSignalsPoints {
         id: trackingSignalsPoints
@@ -87,6 +88,13 @@ Item {
                     if (!trackingSignalsTab.visible)
                         return ;
 
+                    if (emptySeries == null) {
+                        emptySeries = trackingSignalsChart.createSeries(ChartView.SeriesTypeLine, "", trackingSignalsXAxis);
+                        emptySeries.axisYRight = trackingSignalsYAxis;
+                        emptySeries.width = Constants.commonChart.lineWidth;
+                        emptySeries.useOpenGL = Globals.useOpenGL;
+                        trackingSignalsPoints.addEmptySeries(emptySeries);
+                    }
                     if (all_series.length < num_labels) {
                         for (var i = all_series.length; i < num_labels; i++) {
                             var series = trackingSignalsChart.createSeries(ChartView.SeriesTypeLine, trackingSignalsPoints.getLabel(i), trackingSignalsXAxis);
@@ -97,11 +105,6 @@ Item {
                             trackingSignalsPoints.addSeries(series);
                         }
                     }
-                    var series = trackingSignalsChart.createSeries(ChartView.SeriesTypeLine, "", trackingSignalsXAxis);
-                    series.axisYRight = trackingSignalsYAxis;
-                    series.width = Constants.commonChart.lineWidth;
-                    series.useOpenGL = Globals.useOpenGL;
-                    trackingSignalsPoints.addSeries(series);
                     trackingSignalsPoints.fill_all_series();
                     trackingSignalsChart.visible = true;
                     trackingSignalsXAxis.min = trackingSignalsPoints.xaxis_min;

swiftnav_console/tracking_signals_tab.py

@@ -1,7 +1,7 @@
 """Tracking Signals Tab QObjects.
 """
 
-from typing import Dict, List, Any
+from typing import Dict, List, Any, Optional
 
 from PySide2.QtCore import Property, QObject, Signal, Slot
 from PySide2.QtCharts import QtCharts
@@ -24,6 +24,7 @@ class TrackingSignalsPoints(QObject):
     _xaxis_min: float = 0.0
     _xaxis_max: float = 0.0
     _check_labels: List[str] = []
+    _empty_series: Optional[QtCharts.QXYSeries] = None
     _all_series: List[QtCharts.QXYSeries] = []
     _enabled_series: List[QtCharts.QXYSeries] = []
     num_labels_changed = Signal(int, arguments="num_labels")
@@ -76,6 +77,11 @@ def addSeries(self, series) -> None:
         self._all_series.append(series)
         self.all_series_changed.emit()  # type: ignore
 
+    @Slot(QtCharts.QAbstractSeries)  # type: ignore
+    def addEmptySeries(self, series) -> None:
+        """Store a QML created series in empty_series"""
+        self._empty_series = series
+
     @Slot()  # type: ignore
     def fill_all_series(self) -> None:
         cur_num_labels = len(TRACKING_SIGNALS_TAB[Keys.LABELS])
@@ -105,7 +111,10 @@ def fill_all_series(self) -> None:
         for idx, series_points in enumerate(points_for_all_series):
             series = None
             try:
-                series = self._all_series[idx]
+                if idx == len(labels):
+                    series = self._empty_series
+                else:
+                    series = self._all_series[idx]
                 series.clear()
                 series.replace(series_points)
                 series.setName(labels[idx])