Merge pull request numpy#8834 from juliantaylor/count_nonzero-bench

eric-wieser · web-flow · commit 9791f20e113f · 2017-03-25T15:54:41.000Z
BENCH: use initialized memory for count_nonzero benchmark
diff --git a/benchmarks/benchmarks/bench_core.py b/benchmarks/benchmarks/bench_core.py
@@ -137,8 +137,8 @@ class CountNonzero(Benchmark):
     ]
 
     def setup(self, numaxes, size, dtype):
-        self.x = np.empty(shape=(
-            numaxes, size), dtype=dtype)
+        self.x = np.arange(numaxes * size).reshape(numaxes, size)
+        self.x = (self.x % 3).astype(dtype)
 
     def time_count_nonzero(self, numaxes, size, dtype):
         np.count_nonzero(self.x)
diff --git a/doc/source/dev/gitwash/development_workflow.rst b/doc/source/dev/gitwash/development_workflow.rst
@@ -174,6 +174,7 @@ what not to do; the reader has to go look for context elsewhere.
 Standard acronyms to start the commit message with are::
 
    API: an (incompatible) API change
+   BENCH: changes to the benchmark suite
    BLD: change related to building numpy
    BUG: bug fix
    DEP: deprecate something, or remove a deprecated object

Original file line number	Diff line number	Diff line change
`@@ -137,8 +137,8 @@ class CountNonzero(Benchmark):`
`137`	`137`	`]`
`138`	`138`
`139`	`139`	`def setup(self, numaxes, size, dtype):`
`140`		`- self.x = np.empty(shape=(`
`141`		`- numaxes, size), dtype=dtype)`
	`140`	`+ self.x = np.arange(numaxes * size).reshape(numaxes, size)`
	`141`	`+ self.x = (self.x % 3).astype(dtype)`
`142`	`142`
`143`	`143`	`def time_count_nonzero(self, numaxes, size, dtype):`
`144`	`144`	`np.count_nonzero(self.x)`