Skip to content
Snippets Groups Projects
Unverified Commit 41bf0f96 authored by Aleksandr Lesnenko's avatar Aleksandr Lesnenko Committed by GitHub
Browse files

fix tooltip shows data for different X value (#42299)

parent 0f8beaf4
No related branches found
No related tags found
No related merge requests found
Showing
with 10 additions and 3 deletions
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo.png

59.7 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo.png

59.7 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo.png
  • 2-up
  • Swipe
  • Onion skin
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo_Log.png

36.2 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo_Log.png

36.3 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo_Log.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo_Log.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo_Log.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo_Log.png
  • 2-up
  • Swipe
  • Onion skin
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo_Power.png

41.6 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo_Power.png

41.6 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo_Power.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo_Power.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo_Power.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Combo_Power.png
  • 2-up
  • Swipe
  • Onion skin
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Area.png

43.9 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Area.png

43.9 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Area.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Area.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Area.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Area.png
  • 2-up
  • Swipe
  • Onion skin
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Line.png

43 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Line.png

43 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Line.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Line.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Line.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Line.png
  • 2-up
  • Swipe
  • Onion skin
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Normalized_Stacked_Area.png

31.7 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Normalized_Stacked_Area.png

31.7 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Normalized_Stacked_Area.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Normalized_Stacked_Area.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Normalized_Stacked_Area.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Normalized_Stacked_Area.png
  • 2-up
  • Swipe
  • Onion skin
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Normalized_Stacked_Bar.png

26.1 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Normalized_Stacked_Bar.png

26.1 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Normalized_Stacked_Bar.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Normalized_Stacked_Bar.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Normalized_Stacked_Bar.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Normalized_Stacked_Bar.png
  • 2-up
  • Swipe
  • Onion skin
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Stacked_Area.png

43.5 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Stacked_Area.png

43.5 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Stacked_Area.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Stacked_Area.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Stacked_Area.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Multi_Series_Stacked_Area.png
  • 2-up
  • Swipe
  • Onion skin
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Single_Series_Area.png

33.3 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Single_Series_Area.png

33.3 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Single_Series_Area.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Single_Series_Area.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Single_Series_Area.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Single_Series_Area.png
  • 2-up
  • Swipe
  • Onion skin
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Single_Series_Line.png

30.6 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Single_Series_Line.png

30.6 KiB | W: | H:

.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Single_Series_Line.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Single_Series_Line.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Single_Series_Line.png
.loki/reference/chrome_laptop_static_viz_ComboChart_Trend_Single_Series_Line.png
  • 2-up
  • Swipe
  • Onion skin
......@@ -244,7 +244,7 @@ export const getNormalizedDatasetTransform = (
// Copy the dimension value
const normalizedDatum: Datum = {
[X_AXIS_DATA_KEY]: datum[X_AXIS_DATA_KEY],
...datum,
};
// Compute normalized values for metrics
......@@ -536,7 +536,10 @@ const interpolateTimeSeriesData = (
for (let i = 0; i < dataset.length; i++) {
const datum = dataset[i];
result.push(datum);
result.push({
...datum,
[ORIGINAL_INDEX_DATA_KEY]: datum[ORIGINAL_INDEX_DATA_KEY] ?? i,
});
if (i === dataset.length - 1) {
break;
......@@ -545,7 +548,7 @@ const interpolateTimeSeriesData = (
const end = parseTimestamp(dataset[i + 1][X_AXIS_DATA_KEY]);
let start = parseTimestamp(datum[X_AXIS_DATA_KEY]);
while (start.add(count, unit).isBefore(end)) {
while (start.add(count, unit).isBefore(end, unit)) {
const interpolatedValue = start.add(count, unit);
result.push({
[X_AXIS_DATA_KEY]: interpolatedValue.toISOString(),
......
......@@ -360,13 +360,17 @@ describe("dataset transform functions", () => {
expect(result).toEqual([
{
[X_AXIS_DATA_KEY]: "A",
dimensionKey: "A",
series1: 1 / 3,
series2: 2 / 3,
unusedSeries: 100,
},
{
[X_AXIS_DATA_KEY]: "B",
dimensionKey: "B",
series1: 3 / 7,
series2: 4 / 7,
unusedSeries: 100,
},
]);
});
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment