From 872139938830b8dd687f32c905e9e173ac96381e Mon Sep 17 00:00:00 2001
From: Ant Zucaro <azucaro@gmail.com>
Date: Sat, 14 Sep 2013 15:50:50 -0400
Subject: [PATCH] Add weapon averages in the accuracy chart tooltip.

---
 xonstat/static/js/weaponCharts.js |  3 +++
 xonstat/views/player.py           | 18 ++++++++++++------
 2 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/xonstat/static/js/weaponCharts.js b/xonstat/static/js/weaponCharts.js
index 8e32893..f1066fa 100644
--- a/xonstat/static/js/weaponCharts.js
+++ b/xonstat/static/js/weaponCharts.js
@@ -96,6 +96,9 @@ var drawAccuracyChart = function(data) {
           return 0;
         }
       })
+      .tooltip(function(key, x, y, e, graph) {
+        return '<h3>' + key + '</h3>' + '<p>' +  y + ' accuracy in game #' + x + ' <br /> ' + data.averages[key]  + '% average</p>';
+      })
       .color(keyColor);
 
     chart.xAxis
diff --git a/xonstat/views/player.py b/xonstat/views/player.py
index 66a8698..ba8cf0d 100644
--- a/xonstat/views/player.py
+++ b/xonstat/views/player.py
@@ -1096,29 +1096,35 @@ def player_weaponstats_data_json(request):
     # zero-valued weapon stat entries for games where a weapon was not
     # used in that game, but was used in another game for the set
     games_to_weapons = {}
-    weapons_used = []
+    weapons_used = {}
+    sum_avgs = {}
     for ws in weapon_stats_raw:
         if ws.game_id not in games_to_weapons:
             games_to_weapons[ws.game_id] = [ws.weapon_cd]
         else:
             games_to_weapons[ws.game_id].append(ws.weapon_cd)
 
-        if ws.weapon_cd not in weapons_used:
-            weapons_used.append(ws.weapon_cd)
+        weapons_used[ws.weapon_cd] = weapons_used.get(ws.weapon_cd, 0) + 1
+        sum_avgs[ws.weapon_cd] = sum_avgs.get(ws.weapon_cd, 0) + float(ws.hit)/float(ws.fired)
 
     for game_id in games_to_weapons.keys():
-        for weapon_cd in set(weapons_used) - set(games_to_weapons[game_id]):
+        for weapon_cd in set(weapons_used.keys()) - set(games_to_weapons[game_id]):
             weapon_stats_raw.append(PlayerWeaponStat(player_id=player_id,
                 game_id=game_id, weapon_cd=weapon_cd))
 
+    # averages for the weapons used in the range
+    avgs = {}
+    for w in weapons_used.keys():
+        avgs[w] = round(sum_avgs[w]/float(weapons_used[w])*100, 2)
 
-    weapon_stats_raw = sorted(weapon_stats_raw, key              = lambda x: x.game_id)
+    weapon_stats_raw = sorted(weapon_stats_raw, key = lambda x: x.game_id)
     games            = sorted(games_to_weapons.keys())
     weapon_stats     = [ws.to_dict() for ws in weapon_stats_raw]
 
     return {
         "weapon_stats": weapon_stats,
-        "weapons_used": weapons_used,
+        "weapons_used": weapons_used.keys(),
         "games": games,
+        "averages": avgs,
     }
 
-- 
2.39.2