Browse Source

Fix for "Double legend and tooltip for HSR #316" (#331)

master
rozetko 6 years ago committed by GitHub
parent
commit
4d98884e14
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 20
      src/panel/graph_panel/graph_ctrl.ts

20
src/panel/graph_panel/graph_ctrl.ts

@ -363,15 +363,13 @@ class GraphCtrl extends MetricsPanelCtrl {
this.dataList = dataList; this.dataList = dataList;
this.loading = true; this.loading = true;
const { from, to } = this.rangeTimestamp; let seriesList = this.processor.getSeriesList({
this.seriesList = this.processor.getSeriesList({
dataList: this.dataList, dataList: this.dataList,
range: this.range, range: this.range,
}); });
this.dataWarning = null; this.dataWarning = null;
const hasSomePoint = this.seriesList.some(s => s.datapoints.length > 0); const hasSomePoint = seriesList.some(s => s.datapoints.length > 0);
if(!hasSomePoint) { if(!hasSomePoint) {
this.dataWarning = { this.dataWarning = {
@ -379,7 +377,7 @@ class GraphCtrl extends MetricsPanelCtrl {
tip: 'No datapoints returned from data query', tip: 'No datapoints returned from data query',
}; };
} else { } else {
for(let series of this.seriesList) { for(let series of seriesList) {
if(series.isOutsideRange) { if(series.isOutsideRange) {
this.dataWarning = { this.dataWarning = {
title: 'Data points outside time range', title: 'Data points outside time range',
@ -387,15 +385,16 @@ class GraphCtrl extends MetricsPanelCtrl {
}; };
break; break;
} }
const from = _.find(series.datapoints, datapoint => datapoint[0] !== null); }
const to = _.findLast(series.datapoints, datapoint => datapoint[0] !== null); // TODO: multiple metrics will be supported
const from = _.find(seriesList[0].datapoints, datapoint => datapoint[0] !== null);
const to = _.findLast(seriesList[0].datapoints, datapoint => datapoint[0] !== null);
this._dataTimerange = {}; this._dataTimerange = {};
if (from !== undefined && to !== undefined) { if (from !== undefined && to !== undefined) {
this._dataTimerange = { from: from[1], to: to[1] }; this._dataTimerange = { from: from[1], to: to[1] };
} }
} }
}
if(this.analyticsController !== undefined) { if(this.analyticsController !== undefined) {
let { from, to } = this.rangeTimestamp; let { from, to } = this.rangeTimestamp;
@ -409,7 +408,7 @@ class GraphCtrl extends MetricsPanelCtrl {
dataList: hsrData, dataList: hsrData,
range: this.range, range: this.range,
}); });
this.seriesList = _.concat(this.seriesList, hsrSeries); seriesList = _.concat(seriesList, hsrSeries);
await this.analyticsController.fetchAnalyticUnitsSegments(from, to); await this.analyticsController.fetchAnalyticUnitsSegments(from, to);
// TODO: make statuses and detection spans connected // TODO: make statuses and detection spans connected
@ -420,8 +419,9 @@ class GraphCtrl extends MetricsPanelCtrl {
this._dataTimerange.from, this._dataTimerange.from,
this._dataTimerange.to this._dataTimerange.to
); );
this.render(this.seriesList);
} }
this.seriesList = seriesList;
this.render();
this.loading = false; this.loading = false;
} }

Loading…
Cancel
Save