From: Steinar H. Gunderson Date: Tue, 19 Nov 2013 23:10:24 +0000 (+0100) Subject: Properly sort the scores as numbers, not strings. Oops. X-Git-Url: https://git.sesse.net/?p=remoteglot;a=commitdiff_plain;h=860c85720480b9dfb1bb4c300ca3e9b04650a336;hp=a42a7b25408acf92a72a4c04f4b577e5c8b38195 Properly sort the scores as numbers, not strings. Oops. --- diff --git a/www/js/remoteglot.js b/www/js/remoteglot.js index e220f4a..3fb0f17 100644 --- a/www/js/remoteglot.js +++ b/www/js/remoteglot.js @@ -198,7 +198,7 @@ var find_nonstupid_moves = function(data, margin) { var best_score = undefined; var pv_score = undefined; for (var move in data.refutation_lines) { - var score = data.refutation_lines[move].score_sort_key; + var score = parseInt(data.refutation_lines[move].score_sort_key); if (move == data.pv_uci[0]) { pv_score = score; } @@ -218,12 +218,12 @@ var find_nonstupid_moves = function(data, margin) { // The PV move will always be first. var moves = []; for (var move in data.refutation_lines) { - var score = data.refutation_lines[move].score_sort_key; + var score = parseInt(data.refutation_lines[move].score_sort_key); if (move != data.pv_uci[0] && best_score - score <= margin) { moves.push(move); } } - moves = moves.sort(function(a, b) { return data.refutation_lines[b].score_sort_key - data.refutation_lines[a].score_sort_key; }); + moves = moves.sort(function(a, b) { return parseInt(data.refutation_lines[b].score_sort_key) - parseInt(data.refutation_lines[a].score_sort_key); }); moves.unshift(data.pv_uci[0]); return moves;