From 1aab9f1f81431f204fcc7ee8e57be5e3b3e7c5ef Mon Sep 17 00:00:00 2001
From: Tyler Goodlet <jgbt@protonmail.com>
Date: Fri, 10 Mar 2023 09:54:32 -0500
Subject: [PATCH] Actually yes, we need to handle empty in-view range..

---
 piker/ui/view_mode.py | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/piker/ui/view_mode.py b/piker/ui/view_mode.py
index 42fefd62..ecb62557 100644
--- a/piker/ui/view_mode.py
+++ b/piker/ui/view_mode.py
@@ -668,18 +668,17 @@ def overlay_viewlists(
                                 f'yref@xref_pin: {yref}\n'
                             )
 
-                    # if we need to handle not-in-view cases?
-                    # mx_scalars = mx_viz.scalars_from_index(xref)
-                    # if mx_scalars is None:
-                    #     continue
-
+                    # XXX: we need to handle not-in-view cases?
+                    # still not sure why or when tf this happens..
+                    mx_scalars = mx_viz.scalars_from_index(xref)
+                    if mx_scalars is None:
+                        continue
                     (
                         i_start,
                         y_ref_major,
                         r_up_from_major_at_xref,
                         r_down_from_major_at_xref,
-                    ) = mx_viz.scalars_from_index(xref)
-                    # ) = mx_scalars
+                    ) = mx_scalars
 
                     if debug_print:
                         print(