From 4161c58bfdb0561801afde60283057427726cd48 Mon Sep 17 00:00:00 2001 From: Clement Rey Date: Thu, 10 Oct 2024 17:54:55 +0200 Subject: [PATCH] dataframe view: propagate changes --- .../src/space_view_class.rs | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/crates/viewer/re_space_view_dataframe/src/space_view_class.rs b/crates/viewer/re_space_view_dataframe/src/space_view_class.rs index cecbbf0a3ef8e..98178d38a6198 100644 --- a/crates/viewer/re_space_view_dataframe/src/space_view_class.rs +++ b/crates/viewer/re_space_view_dataframe/src/space_view_class.rs @@ -143,23 +143,26 @@ mode sets the default time range to _everything_. You can override this in the s SparseFillStrategy::None }; - let view_columns = query_engine.schema_for_view_contents(&view_contents); - let selected_columns = - view_query.apply_column_visibility_to_view_columns(ctx, &view_columns)?; - - let dataframe_query = re_chunk_store::QueryExpression { + let mut dataframe_query = re_chunk_store::QueryExpression { view_contents: Some(view_contents), filtered_index: view_query.timeline(ctx)?, filtered_index_range: Some(view_query.filter_by_range()?), filtered_point_of_view: view_query.filter_by_event()?, sparse_fill_strategy, - selection: selected_columns, + selection: None, // not yet unsupported by the dataframe view filtered_index_values: None, using_index_values: None, + include_semantically_empty_columns: false, + include_indicator_columns: false, + include_tombstone_columns: false, }; + let view_columns = query_engine.schema_for_query(&dataframe_query); + dataframe_query.selection = + view_query.apply_column_visibility_to_view_columns(ctx, &view_columns)?; + let query_handle = query_engine.query(dataframe_query); let hide_column_actions =