[tools] Beef up GC eval scripts

tools/eval_gc_nvp.py

 #!/usr/bin/env python
 #
 # Copyright 2015 the V8 project authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """This script is used to analyze GCTracer's NVP output."""
 
 
 from argparse import ArgumentParser
@@ skipping 56 matching lines @@
         ret.append("  [{0},{1}[: {2}".format(
           str(min_value), str(max_value), self.histogram[i]))
       else:
         if self.fill_empty:
           ret.append("  [{0},{1}[: {2}".format(
             str(min_value), str(max_value), 0))
     return "\n".join(ret)
 
 
 class Category:
-  def __init__(self, key, histogram):
+  def __init__(self, key, histogram, csv):
     self.key = key
     self.values = []
     self.histogram = histogram
+    self.csv = csv
 
   def process_entry(self, entry):
     if self.key in entry:
       self.values.append(float(entry[self.key]))
       if self.histogram:
         self.histogram.add(float(entry[self.key]))
 
   def min(self):
     return min(self.values)
 
   def max(self):
     return max(self.values)
 
   def avg(self):
+    if len(self.values) == 0:
+      return 0.0
     return sum(self.values) / len(self.values)
 
   def __str__(self):
-    ret = [self.key]
-    ret.append("  len: {0}".format(len(self.values)))
-    if len(self.values) > 0:
-      ret.append("  min: {0}".format(min(self.values)))
-      ret.append("  max: {0}".format(max(self.values)))
-      ret.append("  avg: {0}".format(sum(self.values) / len(self.values)))
-      if self.histogram:
-        ret.append(str(self.histogram))
-    return "\n".join(ret)
+    if self.csv:
+      ret = [self.key]
+      ret.append(len(self.values))
+      ret.append(self.min())
+      ret.append(self.max())
+      ret.append(self.avg())
+      ret = [str(x) for x in ret]
+      return ",".join(ret)
+    else:
+      ret = [self.key]
+      ret.append("  len: {0}".format(len(self.values)))
+      if len(self.values) > 0:
+        ret.append("  min: {0}".format(self.min()))
+        ret.append("  max: {0}".format(self.max()))
+        ret.append("  avg: {0}".format(self.avg()))
+        if self.histogram:
+          ret.append(str(self.histogram))
+      return "\n".join(ret)
 
   def __repr__(self):
     return "<Category: {0}>".format(self.key)
 
 
 def make_key_func(cmp_metric):
   def key_func(a):
     return getattr(a, cmp_metric)()
   return key_func
 
@@ skipping 19 matching lines @@
                       action='store_true',
                       help='omit empty histogram buckets')
   parser.add_argument('--no-histogram', dest='histogram',
                       action='store_false', help='do not print histogram')
   parser.set_defaults(histogram=True)
   parser.set_defaults(histogram_omit_empty=False)
   parser.add_argument('--rank', metavar='<no|min|max|avg>',
                       type=str, nargs='?',
                       default="no",
                       help="rank keys by metric (default: no)")
+  parser.add_argument('--csv', dest='csv',
+                      action='store_true', help='provide output as csv')
   args = parser.parse_args()
 
   histogram = None
   if args.histogram:
     bucket_trait = None
     if args.histogram_type == "log2":
       bucket_trait = Log2Bucket(args.log2_histogram_init_bucket)
     else:
       bucket_trait = LinearBucket(args.linear_histogram_granularity)
     histogram = Histogram(bucket_trait, not args.histogram_omit_empty)
 
-  categories = [ Category(key, deepcopy(histogram))
+  categories = [ Category(key, deepcopy(histogram), args.csv)
                  for key in args.keys ]
 
   while True:
     line = stdin.readline()
     if not line:
       break
     obj = split_nvp(line)
     for category in categories:
       category.process_entry(obj)
 
   if args.rank != "no":
     categories = sorted(categories, key=make_key_func(args.rank), reverse=True)
 
   for category in categories:
     print(category)
 
 
 if __name__ == '__main__':
   main()