status-im/consul
2
0
Fork 0
mirror of https://github.com/status-im/consul.git synced 2025-01-22 19:50:36 +00:00
Code Issues Projects Releases Wiki Activity

Switch tomography to median rather than avg

Browse Source
This commit is contained in:
Ross McFarland 2016-05-17 10:43:50 -07:00
parent 811c77c1b2
commit 4062920969
2 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ui/index.html
View File

@ -600,7 +600,7 @@
{{ tomographyGraph tomography 336 }}
<p class="light small">Minimum: {{ tomography.min }}ms</p>
<p class="light small">Average: {{ tomography.avg }}ms</p>
<p class="light small">Median: {{ tomography.median }}ms</p>
<p class="light small">Maximum: {{ tomography.max }}ms</p>
</script>

12
ui/javascripts/app/routes.js
View File

@ -299,7 +299,15 @@ App.NodesShowRoute = App.BaseRoute.extend({
}
});
var min = Math.min.apply(null, distances);
var avg = sum / distances.length;
var n = distances.length;
var halfN = Math.floor(n / 2);
var median;
if (n % 2) {
// odd
median = distances[halfN];
} else {
median = (distances[halfN - 1] + distances[halfN]) / 2;
}
var max = Math.max.apply(null, distances);
// Return a promise hash of the node and nodes
@ -310,7 +318,7 @@ App.NodesShowRoute = App.BaseRoute.extend({
distances: distances,
n: distances.length,
min: parseInt(min * 100) / 100,
avg: parseInt(avg * 100) / 100,
median: parseInt(median * 100) / 100,
max: parseInt(max * 100) / 100
},
node: Ember.$.getJSON(formatUrl(consulHost + '/v1/internal/ui/node/' + params.name, dc.dc, token)).then(function(data) {