zoom the trend and mean lines to just the values that are visible.

inkjet · Mar 28, 2017 · bcb2f46 · bcb2f46
1 parent 87558c5
commit bcb2f46
Showing 1 changed file with 44 additions and 9 deletions.
diff --git a/LogViewer/LogViewer/Controls/SimpleLineChart.xaml.cs b/LogViewer/LogViewer/Controls/SimpleLineChart.xaml.cs
@@ -839,14 +839,31 @@ private void OnAddTrendLine(object sender, RoutedEventArgs e)
             {
                 return;
             }
-            List<Point> points = new List<Point>(from d in this.series.Values select new Point(d.X, d.Y));
+            // only do the visible points we have zoomed into.
+            List<Point> points = new List<Point>();
+
+            DataValue first = null;
+            DataValue last = null;
+
+            foreach (DataValue d in this.series.Values)
+            {
+                Point point = scaleTransform.Transform(new Point(d.X, d.Y));
+                point = zoomTransform.Transform(point);
+                if (point.X >= 0 && point.X <= this.ActualWidth)
+                {
+                    // then it is a visible point.
+                    if (first == null) first = d;
+                    last = d;
+                    points.Add(new Point(d.X, d.Y));
+                }
+            }
+            if (first == null)
+            {
+                return;
+            }
             double a, b; //  y = a + b.x
             MathHelpers.LinearRegression(points, out a, out b);
 
-            DataValue first = this.series.Values.First();
-            DataValue last = this.series.Values.Last();
-
-            // now scale this line to fit the scaled graph
             Point start = new Point(first.X, a + (b * first.X));
             Point end = new Point(last.X, a + (b * last.X));
 
@@ -972,12 +989,30 @@ private void OnAddMeanLine(object sender, RoutedEventArgs e)
             {
                 return;
             }
-            List<double> points = new List<double>(from d in this.series.Values select d.Y);
-            double mean = MathHelpers.Mean(points);
+            // only do the visible points we have zoomed into.
+            List<double> points = new List<double>();
 
-            DataValue first = this.series.Values.First();
-            DataValue last = this.series.Values.Last();
+            DataValue first = null;
+            DataValue last = null;
 
+            foreach (DataValue d in this.series.Values)
+            {
+                Point point = scaleTransform.Transform(new Point(d.X, d.Y));
+                point = zoomTransform.Transform(point);
+                if (point.X >= 0 && point.X <= this.ActualWidth)
+                {
+                    // then it is a visible point.
+                    if (first == null) first = d;
+                    last = d;
+                    points.Add(d.Y);
+                }
+            }
+            if (first == null)
+            {
+                return;
+            }
+            double mean = MathHelpers.Mean(points);
+
             // now scale this line to fit the scaled graph
             Point start = new Point(first.X, mean);
             Point end = new Point(last.X, mean);