self-review

crates/re_data_store/examples/range_components.rs

@@ -1,7 +1,7 @@
 //! Demonstrates usage of [`re_data_store::polars_util::range_components`].
 //!
 //! ```text
-//! POLARS_FMT_MAX_ROWS=100 cargo r -p re_data_store --example range_components
+//! POLARS_FMT_MAX_ROWS=100 cargo r -p re_data_store --all-features --example range_components
 //! ```
 
 use polars_core::prelude::JoinType;

crates/re_data_store/src/polars_util.rs

@@ -85,9 +85,6 @@ pub fn latest_components(
 /// Iterates over the rows of any number of components and their respective cluster keys, all from
 /// the single point-of-view of the `primary` component, returning an iterator of `DataFrame`s.
 ///
-/// An initial dataframe is yielded with the latest-at state at the start of the time range, if
-/// there is any.
-///
 /// The iterator only ever yields dataframes iff the `primary` component has changed.
 /// A change affecting only secondary components will not yield a dataframe.
 ///
@@ -140,13 +137,7 @@ pub fn range_components<'a, const N: usize>(
             join_type,
         );
 
-        if df.as_ref().map_or(false, |df| {
-            // We only care about the initial state if it A) isn't empty and B) contains any data
-            // at all for the primary component.
-            !df.is_empty() && df.column(primary.as_ref()).is_ok()
-        }) {
-            df_latest = Some(df);
-        }
+        df_latest = Some(df);
     }
 
     let primary_col = components

crates/re_query/src/range.rs

@@ -84,9 +84,7 @@ pub fn range_archetype<'a, A: Archetype + 'a, const N: usize>(
             .map(|(_, row_id, cwi)| (row_id, cwi));
         }
 
-        if cwis_latest_raw[primary_col].is_some() {
-            cwis_latest = Some(cwis_latest_raw);
-        }
+        cwis_latest = Some(cwis_latest_raw);
     }
 
     // send the latest-at state before anything else

crates/re_space_view_time_series/src/visualizer_system.rs

@@ -227,7 +227,17 @@ impl TimeSeriesSystem {
             let line_label =
                 same_label(&points).unwrap_or_else(|| data_result.entity_path.to_string());
 
-            self.add_line_segments(&line_label, points);
+            if points.len() == 1 {
+                self.lines.push(PlotSeries {
+                    label: line_label,
+                    color: points[0].attrs.color,
+                    width: 2.0 * points[0].attrs.radius,
+                    kind: PlotSeriesKind::Scatter,
+                    points: vec![(points[0].time, points[0].value)],
+                });
+            } else {
+                self.add_line_segments(&line_label, points);
+            }
         }
 
         Ok(())

crates/re_viewer/src/ui/visible_history.rs

@@ -246,9 +246,6 @@ fn current_range_ui(
     is_sequence_timeline: bool,
     visible_history: &VisibleHistory,
 ) {
-    let from = visible_history.from(current_time.into());
-    let to = visible_history.to(current_time.into());
-
     let (time_type, quantity_name) = if is_sequence_timeline {
         (TimeType::Sequence, "frame")
     } else {
@@ -260,22 +257,13 @@ fn current_range_ui(
         ctx.app_options.time_zone_for_timestamps,
     );
 
-    if from == to {
-        ui.label(format!(
-            "Showing last data logged on or before {quantity_name} {from_formatted}"
-        ));
-    } else {
-        ui.label(format!(
-            "Showing data between {quantity_name}s {from_formatted} and {}.",
-            time_type.format(
-                visible_history.to(current_time.into()),
-                ctx.app_options.time_zone_for_timestamps
-            )
-        ))
-        .on_hover_text(format!(
-            "This includes the data current as of the starting {quantity_name}."
-        ));
-    };
+    ui.label(format!(
+        "Showing data between {quantity_name}s {from_formatted} and {} (included).",
+        time_type.format(
+            visible_history.to(current_time.into()),
+            ctx.app_options.time_zone_for_timestamps
+        )
+    ));
 }
 
 #[allow(clippy::too_many_arguments)]