Skip to content

Commit

Permalink
increase min samples for benchmark suite (#116)
Browse files Browse the repository at this point in the history
  • Loading branch information
misterdjules authored and delvedor committed Feb 13, 2019
1 parent ed41cba commit 822b0bf
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions bench.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,14 @@
'use strict'

const Benchmark = require('benchmark')
// The default number of samples for Benchmark seems to be low enough that it
// can generate results with significant variance (~2%) for this benchmark
// suite. This makes it sometimes a bit confusing to actually evaluate impact of
// changes on performance. Setting the minimum of samples to 500 results in
// significantly lower variance on my local setup for this tests suite, and
// gives me higher confidence in benchmark results.
Benchmark.options.minSamples = 500

const suite = Benchmark.Suite()

const FindMyWay = require('./')
Expand Down

0 comments on commit 822b0bf

Please sign in to comment.