Skip to content

Commit

Permalink
pallet benchmarking: Bump default steps and repeat (paritytech#14408)
Browse files Browse the repository at this point in the history
* Bump default steps and repeats to useful values

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Add check that scheduler weight depends on component

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Add test to whitelist pallet

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Add env var to disable checks

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* Fix

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>

* remove tests

---------

Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
  • Loading branch information
ggwpez authored and nathanwhit committed Jul 19, 2023
1 parent 423e34c commit 48edacc
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions utils/frame/benchmarking-cli/src/pallet/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ pub struct PalletCmd {
pub extrinsic: Option<String>,

/// 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.
Expand All @@ -55,7 +55,7 @@ pub struct PalletCmd {
pub highest_range_values: Vec<u32>,

/// 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.
Expand Down

0 comments on commit 48edacc

Please sign in to comment.