From 4cc88c8cc72f066ec6d1255c5d873bbd96add905 Mon Sep 17 00:00:00 2001
From: Paul Hauner <paul@paulhauner.com>
Date: Fri, 1 Feb 2019 18:22:43 +1100
Subject: [PATCH] Tidy benches

---
 .../test_harness/benches/state_transition.rs  | 20 ++++---------------
 1 file changed, 4 insertions(+), 16 deletions(-)

diff --git a/beacon_node/beacon_chain/test_harness/benches/state_transition.rs b/beacon_node/beacon_chain/test_harness/benches/state_transition.rs
index 15184e10c9..013ecfd1e1 100644
--- a/beacon_node/beacon_chain/test_harness/benches/state_transition.rs
+++ b/beacon_node/beacon_chain/test_harness/benches/state_transition.rs
@@ -1,11 +1,11 @@
 use criterion::Criterion;
 use criterion::{black_box, criterion_group, criterion_main, Benchmark};
-use env_logger::{Builder, Env};
+// use env_logger::{Builder, Env};
 use test_harness::BeaconChainHarness;
 use types::{ChainSpec, Hash256};
 
 fn mid_epoch_state_transition(c: &mut Criterion) {
-    Builder::from_env(Env::default().default_filter_or("debug")).init();
+    // Builder::from_env(Env::default().default_filter_or("debug")).init();
 
     let validator_count = 1000;
     let mut rig = BeaconChainHarness::new(ChainSpec::foundation(), validator_count);
@@ -56,25 +56,13 @@ fn epoch_boundary_state_transition(c: &mut Criterion) {
                 ))
             })
         })
-        .sample_size(5),
+        .sample_size(5), // sample size is low because function is sloooow.
     );
-
-    /*
-    c.bench_function("mid-epoch state transition 10k validators", move |b| {
-        let state = state.clone();
-        b.iter(|| {
-            let mut state = state.clone();
-            black_box(black_box(
-                state.per_slot_processing(Hash256::zero(), &rig.spec),
-            ))
-        })
-    });
-    */
 }
 
 criterion_group!(
     benches,
-    // mid_epoch_state_transition,
+    mid_epoch_state_transition,
     epoch_boundary_state_transition
 );
 criterion_main!(benches);