Cleaned up profiler code

Author: alexhanh

profile.py

@@ -7,6 +7,7 @@
 from sc2reader import Replay
 
 def parse_replays():
+    # Run four times to dampen noise
     for run in range(1,4):
         file_list = []
         rootdir = "test_replays/build17811/"
@@ -20,45 +21,18 @@ def parse_replays():
             print file
             replay = Replay(file)
 
-# Problem with the profiler is that it adds conciderable amount of overhead
-# cProfile.run("parse_replays()","replay_profile")
-# stats = Stats("replay_profile")
-# stats.strip_dirs().sort_stats("time").print_stats(30)
+# Use the results of this function when comparing performance with other libraries.
+def benchmark_with_timetime():
+    start = time.time()
+    parse_replays()
+    diff = time.time() - start
+    print diff
 
-start = time.time()
-parse_replays()
-diff = time.time() - start
-print diff
+def profile():
+    cProfile.run("parse_replays()","replay_profile")
+    stats = Stats("replay_profile")
+    stats.strip_dirs().sort_stats("time").print_stats(30)  
 
-# ========================================
-# Results for March 1 2011
-# With cProfile
 
-# Tue Mar  1 06:25:22 2011    replay_profile
-# 
-#          20568477 function calls (20565106 primitive calls) in 37.650 seconds
-# 
-#    Ordered by: internal time
-#    List reduced from 184 to 15 due to restriction <15>
-# 
-#    ncalls  tottime  percall  cumtime  percall filename:lineno(function)
-#   3484431    7.903    0.000    8.559    0.000 utils.py:42(get_big)
-#   3057363    5.960    0.000   13.308    0.000 utils.py:89(get_big_int)
-#        33    4.828    0.146   35.395    1.073 parsers.py:225(load)
-#    472272    2.948    0.000    3.360    0.000 objects.py:98(__init__)
-#     42861    2.227    0.000    6.391    0.000 eventparsers.py:154(load)
-#    472272    2.222    0.000    3.223    0.000 parsers.py:254(get_parser)
-#    114174    1.225    0.000    4.265    0.000 eventparsers.py:50(load)
-#    294732    1.224    0.000    3.983    0.000 eventparsers.py:394(load)
-#    473916    1.060    0.000    3.568    0.000 utils.py:108(get_timestamp)
-#   1142493    0.926    0.000    0.926    0.000 utils.py:66(peek)
-#   3965823    0.745    0.000    0.745    0.000 {len}
-#      7095    0.705    0.000    0.705    0.000 {method 'upper' of 'str' objects}
-#    695461    0.644    0.000    0.644    0.000 {range}
-#         1    0.575    0.575   37.571   37.571 profile.py:9(parse_replays)
-#    566628    0.491    0.000    0.491    0.000 {method 'rjust' of 'str' objects}
-
-
-# With time.time()
-# 28.2304489613
-# ========================================
+benchmark_with_timetime()
+#profile()

test_replays/test_all.py

@@ -4,7 +4,6 @@
 import os, sys
 import pytest
 
-#sys.path.append(os.path.normpath(os.path.join(os.path.dirname(os.path.abspath(__file__)),"../")))
 sys.path.insert(0, os.path.normpath(os.path.join(os.path.dirname(os.path.abspath(__file__)),"../")))
 
 from sc2reader import Replay
@@ -122,7 +121,7 @@ def test_ffa():
     # Unknown result is the best we can do.
     print replay.length
     print replay.events[-2]
-    assert replay.winner_known == True
+    assert replay.winner_known == False
 
 def test_unknown_winner():
     replay = Replay("test_replays/build17811/10.SC2Replay")
@@ -148,7 +147,6 @@ def test_us_realm():
     assert replay.player['ShadesofGray'].url == "http://us.battle.net/sc2/en/profile/2358439/1/ShadesofGray/"
     assert replay.player['reddawn'].url == "http://us.battle.net/sc2/en/profile/2198663/1/reddawn/"
 
-# We can be really proud: phpsc2replay fails with the messages!
 def test_kr_realm_and_tampered_messages():
     replay = Replay("test_replays/build17811/11.SC2Replay")
     assert replay.player['명지대학교'].url == "http://kr.battle.net/sc2/en/profile/258945/1/명지대학교/"