stats - stat - prevent 0ms timeDiff breaking movingAverage

libp2p · Apr 25, 2019 · 8c922ea · 8c922ea
1 parent c73efba
commit 8c922ea
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/stats/stat.js b/src/stats/stat.js
@@ -160,7 +160,7 @@ class Stats extends EventEmitter {
    * @returns {void}
    */
   _updateFrequency (latestTime) {
-    const timeDiff = latestTime - this._frequencyLastTime
+    const timeDiff = (latestTime - this._frequencyLastTime) || 1
 
     Object.keys(this._stats).forEach((key) => {
       this._updateFrequencyFor(key, timeDiff, latestTime)
@@ -182,6 +182,8 @@ class Stats extends EventEmitter {
   _updateFrequencyFor (key, timeDiffMS, latestTime) {
     const count = this._frequencyAccumulators[key] || 0
     this._frequencyAccumulators[key] = 0
+    // if `timeDiff` is zero, `hz` becomes Infinity, so we fallback to 1ms
+    const safeTimeDiff = timeDiffMS || 1
     const hz = (count / timeDiffMS) * 1000
 
     let movingAverages = this._movingAverages[key]