From 14ea1f777573906868c307052dc4630fff7b3a8a Mon Sep 17 00:00:00 2001 From: Colin Alworth Date: Fri, 10 Jan 2025 14:06:58 -0600 Subject: [PATCH] fix: Chart subscriptions shouldn't reverse positions Fixes DH-18389 --- .../java/io/deephaven/web/client/api/widget/plot/ChartData.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/client-api/src/main/java/io/deephaven/web/client/api/widget/plot/ChartData.java b/web/client-api/src/main/java/io/deephaven/web/client/api/widget/plot/ChartData.java index bd08d045bf2..e0cb3393bc6 100644 --- a/web/client-api/src/main/java/io/deephaven/web/client/api/widget/plot/ChartData.java +++ b/web/client-api/src/main/java/io/deephaven/web/client/api/widget/plot/ChartData.java @@ -72,7 +72,7 @@ public void update(AbstractTableSubscription.SubscriptionEventData tableData) { private void replaceDataRange(SubscriptionTableData tableData, Range positions) { RangeSet keys = tableData.getFullIndex().getRange() - .subsetForPositions(RangeSet.ofRange(positions.getFirst(), positions.getLast()), true); + .subsetForPositions(RangeSet.ofRange(positions.getFirst(), positions.getLast()), false); // we don't touch the indexes at all, only need to walk each column and replace values in this range for (Entry, JsArray>> columnMap : cachedData.entrySet()) { Column col = table.findColumn(columnMap.getKey());