From 0a7e16946567bd2f85193943ec6d355f9ee4b437 Mon Sep 17 00:00:00 2001
From: lyxia <lysxia@gmail.com>
Date: Wed, 15 Nov 2017 16:21:43 -0500
Subject: [PATCH 1/4] Add option for full unit name

---
 site/index.html          |  6 +++---
 site/js/gipeda.js        |  2 ++
 src/BenchNames.hs        | 15 ++++++++++-----
 src/BenchmarkSettings.hs |  5 ++++-
 src/ReportTypes.hs       |  1 +
 5 files changed, 20 insertions(+), 9 deletions(-)

diff --git a/site/index.html b/site/index.html
index 2db48c7..b9df62e 100644
--- a/site/index.html
+++ b/site/index.html
@@ -352,7 +352,7 @@ <h1>Benchmarks</h1>
        <div class="panel-heading" role="tab" id="heading-{{@index}}">
         <h4 class="panel-title">
          <a class="accordion-toggle" data-toggle="collapse" href="#table-{{@index}}">
-         {{groupName}}
+         {{groupName}}{{#if unitFull}}, {{unitFull}}{{/if}}
          </a>
         </h4>
        </div>
@@ -471,7 +471,7 @@ <h2>
        <div class="panel-heading" role="tab" id="heading-{{@index}}">
         <h4 class="panel-title">
          <a class="accordion-toggle" data-toggle="collapse" href="#table-{{@index}}">
-         {{groupName}}
+         {{groupName}}{{#if unitFull}}, {{unitFull}}{{/if}}
          <span class="stats pull-right">
           {{> summary-icons groupStats}}
           <span class="indicator-toggled glyphicon glyphicon-chevron-down text-grey"/>
@@ -579,7 +579,7 @@ <h2>
        <div class="panel-heading" role="tab" id="heading-{{@index}}">
         <h4 class="panel-title">
          <a class="accordion-toggle" data-toggle="collapse" href="#table-{{@index}}">
-         {{groupName}}
+         {{groupName}}{{#if unitFull}}, {{unitFull}}{{/if}}
          <span class="stats pull-right">
           {{> summary-icons groupStats}}
           <span class="indicator-toggled glyphicon glyphicon-chevron-down text-grey"/>
diff --git a/site/js/gipeda.js b/site/js/gipeda.js
index 2f42d96..c31442f 100644
--- a/site/js/gipeda.js
+++ b/site/js/gipeda.js
@@ -466,6 +466,8 @@ function calculate_groups(rev1, rev2) {
       }).filter(function (br) {return br;});
       return {
 	groupName: group.groupName || "Benchmarks",
+    unit: group.groupUnit,
+    unitFull: group.groupUnitFull,
 	benchResults: benchmarks,
 	groupStats: groupStats(benchmarks),
       };
diff --git a/src/BenchNames.hs b/src/BenchNames.hs
index 5153797..2168a8c 100644
--- a/src/BenchNames.hs
+++ b/src/BenchNames.hs
@@ -3,6 +3,7 @@ module BenchNames where
 
 import qualified Data.ByteString.Lazy as BS
 import Data.Aeson
+import Control.Applicative
 import Control.Monad
 import qualified Data.Map as M
 import qualified Data.Text as T
@@ -19,12 +20,16 @@ benchNamesMain benchNames = do
     settings <- S.readSettings "gipeda.yaml"
 
     let groups =
-            map (uncurry BenchGroup) $
-            sort $
+            sortOn (liftA2 (,) groupName groupMembers) $
+            map (\(name, (s, bns)) -> BenchGroup
+              { groupName = name
+              , groupMembers = bns
+              , groupUnitFull = S.unitFull s
+              }) $
             M.toList $
-            M.map sort $
-            M.fromListWith (++)
-                [ (S.group s, [bn])
+            M.map (fmap sort) $
+            M.fromListWith (\(s, bns) (_, bns') -> (s, bns ++ bns'))
+                [ (S.group s, (s, [bn]))
                 | bn <- benchNames
                 , let s = S.benchSettings settings bn
                 ]
diff --git a/src/BenchmarkSettings.hs b/src/BenchmarkSettings.hs
index a7a5a45..1a9fe8b 100644
--- a/src/BenchmarkSettings.hs
+++ b/src/BenchmarkSettings.hs
@@ -24,6 +24,7 @@ type BenchName = String
 data BenchSettings = BenchSettings
     { smallerIsBetter :: Bool
     , unit :: String
+    , unitFull :: Maybe String
     , numberType :: NumberType
     , group :: String
     , threshold :: Double
@@ -33,7 +34,7 @@ data BenchSettings = BenchSettings
 instance ToJSON BenchSettings
 
 defaultBenchSettings :: BenchSettings
-defaultBenchSettings = BenchSettings True "" IntegralNT "" 3 True
+defaultBenchSettings = BenchSettings True "" Nothing IntegralNT "" 3 True
 
 newtype S = S { unS :: BenchName -> BenchSettings }
 newtype SM = SM (BenchName -> (BenchSettings -> BenchSettings))
@@ -60,6 +61,7 @@ instance FromJSON SM where
         m <- o .: "match"
         mt <- o .:? "type"
         mu <- o .:? "unit"
+        muf <- o.:? "unitFull"
         mg <- o .:? "group"
         ms <- o .:? "smallerIsBetter"
         mth <- o .:? "threshold"
@@ -68,6 +70,7 @@ instance FromJSON SM where
             if n `matches` m then
                b { numberType      = fromMaybe (numberType b) mt
                  , unit            = fromMaybe (unit b) mu
+                 , unitFull        = muf
                  , group           = fromMaybe (group b) mg
                  , smallerIsBetter = fromMaybe (smallerIsBetter b) ms
                  , threshold       = fromMaybe (threshold b) mth
diff --git a/src/ReportTypes.hs b/src/ReportTypes.hs
index 5f60463..51160d5 100644
--- a/src/ReportTypes.hs
+++ b/src/ReportTypes.hs
@@ -109,6 +109,7 @@ instance FromJSON ChangeType
 data BenchGroup = BenchGroup
     { groupName :: String
     , groupMembers :: [BenchName]
+    , groupUnitFull :: Maybe String
     }
  deriving (Generic)
 instance ToJSON BenchGroup

From 2408aa3acd4e7d22251658de4647f6dcbeebab82 Mon Sep 17 00:00:00 2001
From: lyxia <lysxia@gmail.com>
Date: Wed, 15 Nov 2017 21:37:32 -0500
Subject: [PATCH 2/4] Clean rendering of values

---
 site/js/gipeda.js | 37 +++++++++++++++++++++++++++++++++----
 1 file changed, 33 insertions(+), 4 deletions(-)

diff --git a/site/js/gipeda.js b/site/js/gipeda.js
index c31442f..2e12be9 100644
--- a/site/js/gipeda.js
+++ b/site/js/gipeda.js
@@ -358,6 +358,7 @@ function setting_for(name) {
     return benchSettings;
 }
 
+var unitPrefix = [["G", 9], ["M", 6], ["k", 3], ["", 0], ["m", -3], ["u", -6], ["n", -9]]
 
 // The following logic should be kept in sync with toResult in ReportTypes.hs
 function compareResults (res1, res2) {
@@ -366,11 +367,27 @@ function compareResults (res1, res2) {
     var name = res1? res1.name : res2.name;
     var s = setting_for(name);
 
+    var value1 = res1 ? res1.value : null;
+    var value2 = res2 ? res2.value : null;
+    var unit = s.unit;
+
+    for (var i = 0; i < unitPrefix.length; i++) {
+        var magnitude = Math.pow(10, unitPrefix[i][1]);
+        if (i == unitPrefix.length - 1
+          || ( (value1 === null || value1 > magnitude)
+            && (value2 === null || value2 > magnitude))) {
+          if (value1 !== null) { value1 = (value1 / magnitude).toFixed(2); }
+          if (value2 !== null) { value2 = (value2 / magnitude).toFixed(2); }
+          unit = unitPrefix[i][0] + unit;
+          break;
+        }
+    }
+
     var res = {
         name: name,
-        previous:    res1 ? res1.value : null,
-        value:       res2 ? res2.value : null,
-        unit:        s.unit,
+        previous:    value1,
+        value:       value2,
+        unit:        unit,
         important:   s.important,
         changeType: "Boring",
         change: "foobar",
@@ -434,10 +451,22 @@ function singleResult (res) {
     var name = res.name;
     var s = setting_for(name);
 
+    var value = res.value;
+    var unit = s.unit;
+
+    for (var i = 0; i < unitPrefix.length; i++) {
+        var magnitude = Math.pow(10, unitPrefix[i][1]);
+        if (i == unitPrefix.length - 1 || value > magnitude) {
+          value /= magnitude;
+          unit = unitPrefix[i][0] + unit;
+          break;
+        }
+    }
+
     return  {
         name: name,
         previous:    null,
-        value:       res.value,
+        value:       value.toFixed(2),
         unit:        s.unit,
         important:   s.important,
         changeType: "Boring",

From 6bcf67cd0dd920a516bcaa9227442bf347b8cecb Mon Sep 17 00:00:00 2001
From: lyxia <lysxia@gmail.com>
Date: Wed, 15 Nov 2017 21:47:33 -0500
Subject: [PATCH 3/4] Reorder columns

---
 site/index.html | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/site/index.html b/site/index.html
index b9df62e..f373d3b 100644
--- a/site/index.html
+++ b/site/index.html
@@ -488,9 +488,9 @@ <h4 class="panel-title">
           <tr>
           <th class="col-xs-5">Benchmark name</th>
           <th class="col-xs-2 text-right">previous</th>
-          <th class="col-xs-2 text-right">change</th>
           <th class="col-xs-2 text-right">now</th>
           <th class="col-xs-1 text-left"></th>
+          <th class="col-xs-2 text-right">change</th>
           </tr>
           </thead>
           <tbody>
@@ -507,9 +507,9 @@ <h4 class="panel-title">
 	     </a>
 	    </td>
             <td class="text-right">{{previous}}</td>
-            <td class="text-right">{{change}}</td>
             <td class="text-right">{{value}}</td>
             <td class="text-left">{{unit}}</td>
+            <td class="text-right">{{change}}</td>
             </tr>
            {{/each}}
           </tbody>
@@ -596,9 +596,9 @@ <h4 class="panel-title">
           <tr>
           <th class="col-xs-5">Benchmark name</th>
           <th class="col-xs-2 text-right">previous</th>
-          <th class="col-xs-2 text-right">change</th>
           <th class="col-xs-2 text-right">now</th>
           <th class="col-xs-1 text-right"></th>
+          <th class="col-xs-2 text-right">change</th>
           </tr>
           </thead>
           <tbody>
@@ -615,9 +615,9 @@ <h4 class="panel-title">
 	     </a>
 	    </td>
             <td class="text-right">{{previous}}</td>
-            <td class="text-right">{{change}}</td>
             <td class="text-right">{{value}}</td>
             <td class="text-left">{{unit}}</td>
+            <td class="text-right">{{change}}</td>
             </tr>
            {{/each}}
           </tbody>

From 2f9bb0b79b3651b71735554623dd69762ce47aa6 Mon Sep 17 00:00:00 2001
From: lyxia <lysxia@gmail.com>
Date: Sat, 18 Nov 2017 12:11:24 -0500
Subject: [PATCH 4/4] Add option for rescaling values

---
 site/js/gipeda.js        | 71 +++++++++++++++++++++-------------------
 src/BenchmarkSettings.hs |  6 +++-
 2 files changed, 43 insertions(+), 34 deletions(-)

diff --git a/site/js/gipeda.js b/site/js/gipeda.js
index 2e12be9..4210897 100644
--- a/site/js/gipeda.js
+++ b/site/js/gipeda.js
@@ -343,6 +343,8 @@ function setting_for(name) {
     var benchSettings = {
         smallerIsBetter: true,
         unit: "",
+        unitFull: "",
+        rescalable: false,
         type: "integral",
         group: "",
         threshold: 3,
@@ -360,34 +362,43 @@ function setting_for(name) {
 
 var unitPrefix = [["G", 9], ["M", 6], ["k", 3], ["", 0], ["m", -3], ["u", -6], ["n", -9]]
 
-// The following logic should be kept in sync with toResult in ReportTypes.hs
-function compareResults (res1, res2) {
-    if (!res1 && !res2) { return; }
-
-    var name = res1? res1.name : res2.name;
-    var s = setting_for(name);
-
-    var value1 = res1 ? res1.value : null;
-    var value2 = res2 ? res2.value : null;
-    var unit = s.unit;
+// Make numbers more human-readable.
+function rescale(s, res) {
+    // Only rescale nonzero rescalable numbers.
+    if (!s.rescalable || (!res.previous && !res.value)) { return; }
 
+    // If the result is a comparison, i.e., contains both res.previous and
+    // res.value, then both are rescaled together.
     for (var i = 0; i < unitPrefix.length; i++) {
         var magnitude = Math.pow(10, unitPrefix[i][1]);
-        if (i == unitPrefix.length - 1
-          || ( (value1 === null || value1 > magnitude)
-            && (value2 === null || value2 > magnitude))) {
-          if (value1 !== null) { value1 = (value1 / magnitude).toFixed(2); }
-          if (value2 !== null) { value2 = (value2 / magnitude).toFixed(2); }
-          unit = unitPrefix[i][0] + unit;
+        if (i == unitPrefix.length - 1  // Fallback to nano
+          || ( (res.previous === null || res.previous >= magnitude)
+            && (res.value === null || res.value >= magnitude))) {
+          if (res.previous !== null) { res.previous = res.previous / magnitude; }
+          if (res.value !== null) { res.value = res.value / magnitude; }
+          res.unit = unitPrefix[i][0] + s.unit;
           break;
         }
     }
 
+    if (s.type == "float" || magnitude > 1) {
+      if (res.previous !== null) { res.previous = res.previous.toFixed(2); }
+      if (res.value !== null) { res.value = res.value.toFixed(2); }
+    }
+}
+
+// The following logic should be kept in sync with toResult in ReportTypes.hs
+function compareResults (res1, res2) {
+    if (!res1 && !res2) { return; }
+
+    var name = res1? res1.name : res2.name;
+    var s = setting_for(name);
+
     var res = {
         name: name,
-        previous:    value1,
-        value:       value2,
-        unit:        unit,
+        previous:    res1 ? res1.value : null,
+        value:       res2 ? res2.value : null,
+        unit:        s.unit,
         important:   s.important,
         changeType: "Boring",
         change: "foobar",
@@ -442,6 +453,8 @@ function compareResults (res1, res2) {
         }
     }
 
+    rescale(s, res);
+
     return res;
 }
 
@@ -451,27 +464,19 @@ function singleResult (res) {
     var name = res.name;
     var s = setting_for(name);
 
-    var value = res.value;
-    var unit = s.unit;
-
-    for (var i = 0; i < unitPrefix.length; i++) {
-        var magnitude = Math.pow(10, unitPrefix[i][1]);
-        if (i == unitPrefix.length - 1 || value > magnitude) {
-          value /= magnitude;
-          unit = unitPrefix[i][0] + unit;
-          break;
-        }
-    }
-
-    return  {
+    var newRes = {
         name: name,
         previous:    null,
-        value:       value.toFixed(2),
+        value:       res.value,
         unit:        s.unit,
         important:   s.important,
         changeType: "Boring",
         change:     "",
     };
+
+    rescale(s, newRes);
+
+    return newRes;
 }
 
 // Some views require the data to be prepared in ways that are 
diff --git a/src/BenchmarkSettings.hs b/src/BenchmarkSettings.hs
index 1a9fe8b..dd204a9 100644
--- a/src/BenchmarkSettings.hs
+++ b/src/BenchmarkSettings.hs
@@ -25,6 +25,7 @@ data BenchSettings = BenchSettings
     { smallerIsBetter :: Bool
     , unit :: String
     , unitFull :: Maybe String
+    , rescalable :: Bool
     , numberType :: NumberType
     , group :: String
     , threshold :: Double
@@ -33,8 +34,9 @@ data BenchSettings = BenchSettings
     deriving (Show, Generic)
 instance ToJSON BenchSettings
 
+-- Keep in sync with setting_for() in gipeda.js
 defaultBenchSettings :: BenchSettings
-defaultBenchSettings = BenchSettings True "" Nothing IntegralNT "" 3 True
+defaultBenchSettings = BenchSettings True "" Nothing False IntegralNT "" 3 True
 
 newtype S = S { unS :: BenchName -> BenchSettings }
 newtype SM = SM (BenchName -> (BenchSettings -> BenchSettings))
@@ -62,6 +64,7 @@ instance FromJSON SM where
         mt <- o .:? "type"
         mu <- o .:? "unit"
         muf <- o.:? "unitFull"
+        mr <- o .:? "rescalable"
         mg <- o .:? "group"
         ms <- o .:? "smallerIsBetter"
         mth <- o .:? "threshold"
@@ -71,6 +74,7 @@ instance FromJSON SM where
                b { numberType      = fromMaybe (numberType b) mt
                  , unit            = fromMaybe (unit b) mu
                  , unitFull        = muf
+                 , rescalable      = fromMaybe (rescalable b) mr
                  , group           = fromMaybe (group b) mg
                  , smallerIsBetter = fromMaybe (smallerIsBetter b) ms
                  , threshold       = fromMaybe (threshold b) mth