Merge pull request #31 from lologarithm/fix_oomavg

v1.1.6 - Updated api impl to not return NaN for oomat when no sims oom'd
lologarithm · Aug 22, 2021 · 06e2730 · 06e2730
2 parents 5917f9c + 346ff18
commit 06e2730
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/tbc/api_impl.go b/tbc/api_impl.go
@@ -260,7 +260,9 @@ func (aggregator *MetricsAggregator) getResult() SimResult {
 	result.DpsHist = aggregator.dpsHist
 
 	result.NumOom = aggregator.numOom
-	result.OomAtAvg = aggregator.oomAtSum / float64(aggregator.numOom)
+	if result.NumOom > 0 {
+		result.OomAtAvg = aggregator.oomAtSum / float64(aggregator.numOom)
+	}
 	result.DpsAtOomAvg = aggregator.dpsAtOomSum / numSims
 
 	result.Casts = aggregator.casts

diff --git a/ui/index.html b/ui/index.html
@@ -287,7 +287,7 @@ <h4>Standard Elemental Build</h4>
         <div id="pullout">
             <div class="dtl" style="height:100%;width:100%;padding:10px">
                 <div id="themetoggle" style="height:20px; bottom: 10px; cursor: pointer;" onclick="toggletheme()">
-                    <text>v1.1.5</text>
+                    <text>v1.1.6</text>
                     <img id="themebulb" style="height: 20px; width: 20px" src="icons/light-bulb.svg"></img>
                 </div>
                 <div style="margin-top:10px">

diff --git a/ui/lib.wasm b/ui/lib.wasm