From 116b6e65dcda0e5897139e35950206e8d6fb9070 Mon Sep 17 00:00:00 2001 From: Oliver Tale-Yazdi Date: Sun, 18 Jun 2023 13:50:34 +0200 Subject: [PATCH] pallet benchmarking: Bump default steps and repeat (#14408) * Bump default steps and repeats to useful values Signed-off-by: Oliver Tale-Yazdi * Add check that scheduler weight depends on component Signed-off-by: Oliver Tale-Yazdi * Add test to whitelist pallet Signed-off-by: Oliver Tale-Yazdi * Add env var to disable checks Signed-off-by: Oliver Tale-Yazdi * Fix Signed-off-by: Oliver Tale-Yazdi * remove tests --------- Signed-off-by: Oliver Tale-Yazdi --- substrate/utils/frame/benchmarking-cli/src/pallet/mod.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/substrate/utils/frame/benchmarking-cli/src/pallet/mod.rs b/substrate/utils/frame/benchmarking-cli/src/pallet/mod.rs index b47cb156ec..ca87ddebb3 100644 --- a/substrate/utils/frame/benchmarking-cli/src/pallet/mod.rs +++ b/substrate/utils/frame/benchmarking-cli/src/pallet/mod.rs @@ -43,7 +43,7 @@ pub struct PalletCmd { pub extrinsic: Option, /// Select how many samples we should take across the variable components. - #[arg(short, long, default_value_t = 2)] + #[arg(short, long, default_value_t = 50)] pub steps: u32, /// Indicates lowest values for each of the component ranges. @@ -55,7 +55,7 @@ pub struct PalletCmd { pub highest_range_values: Vec, /// Select how many repetitions of this benchmark should run from within the wasm. - #[arg(short, long, default_value_t = 1)] + #[arg(short, long, default_value_t = 20)] pub repeat: u32, /// Select how many repetitions of this benchmark should run from the client.