Fix silently interpreting zero time range as latest-at query

crates/re_entity_db/src/lib.rs

@@ -34,8 +34,6 @@ pub(crate) use self::entity_tree::{ClearCascade, CompactedStoreEvents};
 use re_log_types::DataTableError;
 pub use re_log_types::{EntityPath, EntityPathPart, TimeInt, Timeline};
 
-pub use re_query::{ExtraQueryHistory, VisibleHistory, VisibleHistoryBoundary};
-
 #[cfg(feature = "serde")]
 pub use blueprint::components::EntityPropertiesComponent;
 #[cfg(feature = "serde")]

crates/re_log_types/src/time_point/mod.rs

@@ -128,7 +128,12 @@ impl TimeType {
     }
 
     #[inline]
-    pub fn format(&self, time_int: TimeInt, time_zone_for_timestamps: TimeZone) -> String {
+    pub fn format(
+        &self,
+        time_int: impl Into<TimeInt>,
+        time_zone_for_timestamps: TimeZone,
+    ) -> String {
+        let time_int = time_int.into();
         match time_int {
             TimeInt::STATIC => "<static>".into(),
             // TODO(#5264): remove time panel hack once we migrate to the new static UI

crates/re_log_types/src/time_range.rs

@@ -101,6 +101,22 @@ impl TimeRange {
             max: self.max.max(other.max),
         }
     }
+
+    pub fn from_visible_time_range(
+        range: &re_types_core::datatypes::VisibleTimeRange,
+        cursor: impl Into<re_types_core::datatypes::TimeInt>,
+    ) -> Self {
+        let cursor = cursor.into();
+
+        let mut min = range.start.start_boundary_time(cursor);
+        let mut max = range.end.end_boundary_time(cursor);
+
+        if min > max {
+            std::mem::swap(&mut min, &mut max);
+        }
+
+        Self::new(min, max)
+    }
 }
 
 impl re_types_core::SizeBytes for TimeRange {

crates/re_query/src/lib.rs

@@ -6,7 +6,6 @@ mod flat_vec_deque;
 mod latest_at;
 mod promise;
 mod range;
-mod visible_history;
 
 pub mod clamped_zip;
 pub mod range_zip;
@@ -19,7 +18,6 @@ pub use self::latest_at::{LatestAtComponentResults, LatestAtMonoResult, LatestAt
 pub use self::promise::{Promise, PromiseId, PromiseResolver, PromiseResult};
 pub use self::range::{RangeComponentResults, RangeData, RangeResults};
 pub use self::range_zip::*;
-pub use self::visible_history::{ExtraQueryHistory, VisibleHistory, VisibleHistoryBoundary};
 
 pub(crate) use self::latest_at::LatestAtCache;
 pub(crate) use self::range::{RangeCache, RangeComponentResultsInner};

crates/re_space_view/src/lib.rs

@@ -9,7 +9,6 @@ mod screenshot;
 mod space_view;
 mod space_view_contents;
 mod sub_archetypes; // TODO(andreas): better name before `sub_archetype` sticks around?
-mod visual_time_range;
 mod visualizable;
 
 pub use data_query::{DataQuery, EntityOverrideContext, PropertyResolver};
@@ -21,11 +20,6 @@ pub use sub_archetypes::{
     edit_blueprint_component, entity_path_for_view_property, get_blueprint_component,
     query_view_property, query_view_property_or_default, view_property,
 };
-pub use visual_time_range::{
-    query_visual_history, time_range_from_visible_time_range,
-    visible_history_boundary_from_time_range_boundary,
-    visible_history_boundary_to_time_range_boundary,
-};
 pub use visualizable::determine_visualizable_entities;
 
 pub mod external {