CLN: ASV eval benchmark
State: closed

- Added `np.random.seed(1234)` in setup classes where random data is created xref #8144

- Ran flake8 and replaced star imports

- Removed some initialization of variables within the `time_*` methods

```
asv run -b ^eval
[  0.00%] ·· Benchmarking conda-py3.6-Cython-matplotlib-numexpr-numpy-openpyxl-pytables-pytest-scipy-sqlalchemy-xlrd-xlsxwriter-xlwt
[ 14.29%] ··· Running eval.Eval.time_add                                                                                  41.8±0.7ms;...
[ 28.57%] ··· Running eval.Eval.time_and                                                                                  55.5±0.4ms;...
[ 42.86%] ··· Running eval.Eval.time_chained_cmp                                                                            48.7±1ms;...
[ 57.14%] ··· Running eval.Eval.time_mult                                                                                   38.5±1ms;...
[ 71.43%] ··· Running eval.Query.time_query_datetime_column                                                                       19.1ms
[ 85.71%] ··· Running eval.Query.time_query_datetime_index                                                                        51.9ms
[100.00%] ··· Running eval.Query.time_query_with_boolean_selection                                                                66.9ms
```

Labels:
name: Benchmark
description: Performance (ASV) benchmarks

Comments:
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/18500?src=pr&el=h1) Report
> Merging [#18500](https://codecov.io/gh/pandas-dev/pandas/pull/18500?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/38f41e64f4b8a0479f8835022af5e7343ccf8498?src=pr&el=desc) will **increase** coverage by `0.02%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/pandas-dev/pandas/pull/18500/graphs/tree.svg?token=eZ4WkYLtcO&src=pr&width=650&height=150)](https://codecov.io/gh/pandas-dev/pandas/pull/18500?src=pr&el=tree)

```diff
@@            Coverage Diff             @@
##           master   #18500      +/-   ##
==========================================
+ Coverage    91.3%   91.32%   +0.02%     
==========================================
  Files         163      163              
  Lines       49781    49781              
==========================================
+ Hits        45451    45463      +12     
+ Misses       4330     4318      -12
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `89.12% <ø> (+0.02%)` | :arrow_up: |
| #single | `40.72% <ø> (ø)` | :arrow_up: |

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/18500?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/plotting/\_converter.py](https://codecov.io/gh/pandas-dev/pandas/pull/18500?src=pr&el=tree#diff-cGFuZGFzL3Bsb3R0aW5nL19jb252ZXJ0ZXIucHk=) | `65.25% <0%> (+1.81%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/18500?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/18500?src=pr&el=footer). Last update [38f41e6...f6381ea](https://codecov.io/gh/pandas-dev/pandas/pull/18500?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

FYI the random see is *already* defined in ``pandas_vb_common`` I would actually remove it entirely from benchmark suite (except for there). (and put a note in each file that its already defined).

separate PR for this.
@jreback I think the random seed should be included in the `setup` function instead so that the same data is generated for each benchmark. If the seed is global then different random data is generated for each benchmark. 

>You can also include a module-level setup function, which will be run for every benchmark within the module, prior to any setup assigned specifically to each function.


ok then let’s remove from the global name space (or maybe put a comment there) and add to setup as u r already doing
I think the module level setup is a good idea