From 7b46928fc9cda9e5200d72c15253f60981eedc74 Mon Sep 17 00:00:00 2001
From: Dean Lee <deanlee3@gmail.com>
Date: Tue, 22 Nov 2022 02:48:08 +0800
Subject: [PATCH] Cabana: fix auto zoom y-axis for multiple line series
 (#26558)

auto zoom y-axis for multiple series
---
 tools/cabana/chartswidget.cc | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tools/cabana/chartswidget.cc b/tools/cabana/chartswidget.cc
index bc1f1e2a38..3029d9db89 100644
--- a/tools/cabana/chartswidget.cc
+++ b/tools/cabana/chartswidget.cc
@@ -365,10 +365,10 @@ void ChartView::updateAxisY() {
     }
   } else {
     for (auto &s : sigs) {
-      for (int i = 0; i < s.series->count(); ++i) {
-        double y = s.series->at(i).y();
-        if (y < min_y) min_y = y;
-        if (y > max_y) max_y = y;
+      auto begin = std::lower_bound(s.vals.begin(), s.vals.end(), axis_x->min(), [](auto &p, double x) { return p.x() < x; });
+      for (auto it = begin; it != s.vals.end() && it->x() <= axis_x->max(); ++it) {
+        if (it->y() < min_y) min_y = it->y();
+        if (it->y() > max_y) max_y = it->y();
       }
     }
   }