From 60c2506a7145055fab337e5d7491835dbabb7aa1 Mon Sep 17 00:00:00 2001
From: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Date: Mon, 1 Aug 2022 20:56:19 +0200
Subject: [PATCH] Remove `remove_member_wrong_refund` from `phragmen`
 WeightInfo (#11952)

* Remove 'remove_member_wrong_refund' from WeightInfo

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

* ".git/.scripts/bench-bot.sh" pallet dev pallet_elections_phragmen

Co-authored-by: command-bot <>
---
 .../frame/elections-phragmen/src/weights.rs   | 181 +++++++++---------
 1 file changed, 93 insertions(+), 88 deletions(-)

diff --git a/substrate/frame/elections-phragmen/src/weights.rs b/substrate/frame/elections-phragmen/src/weights.rs
index 5ad986bf142..07ee7aa2012 100644
--- a/substrate/frame/elections-phragmen/src/weights.rs
+++ b/substrate/frame/elections-phragmen/src/weights.rs
@@ -18,22 +18,24 @@
 //! Autogenerated weights for pallet_elections_phragmen
 //!
 //! THIS FILE WAS AUTO-GENERATED USING THE SUBSTRATE BENCHMARK CLI VERSION 4.0.0-dev
-//! DATE: 2022-05-23, STEPS: `50`, REPEAT: 20, LOW RANGE: `[]`, HIGH RANGE: `[]`
+//! DATE: 2022-08-01, STEPS: `50`, REPEAT: 20, LOW RANGE: `[]`, HIGH RANGE: `[]`
+//! HOSTNAME: `bm3`, CPU: `Intel(R) Core(TM) i7-7700K CPU @ 4.20GHz`
 //! EXECUTION: Some(Wasm), WASM-EXECUTION: Compiled, CHAIN: Some("dev"), DB CACHE: 1024
 
 // Executed Command:
-// ./target/production/substrate
+// /home/benchbot/cargo_target_dir/production/substrate
 // benchmark
 // pallet
-// --chain=dev
 // --steps=50
 // --repeat=20
-// --pallet=pallet_elections_phragmen
 // --extrinsic=*
 // --execution=wasm
 // --wasm-execution=compiled
-// --template=./.maintain/frame-weight-template.hbs
+// --heap-pages=4096
+// --pallet=pallet_elections_phragmen
+// --chain=dev
 // --output=./frame/elections-phragmen/src/weights.rs
+// --template=./.maintain/frame-weight-template.hbs
 
 #![cfg_attr(rustfmt, rustfmt_skip)]
 #![allow(unused_parens)]
@@ -54,7 +56,6 @@ pub trait WeightInfo {
 	fn renounce_candidacy_runners_up() -> Weight;
 	fn remove_member_without_replacement() -> Weight;
 	fn remove_member_with_replacement() -> Weight;
-	fn remove_member_wrong_refund() -> Weight;
 	fn clean_defunct_voters(v: u32, d: u32, ) -> Weight;
 	fn election_phragmen(c: u32, v: u32, e: u32, ) -> Weight;
 }
@@ -67,10 +68,11 @@ impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> {
 	// Storage: Elections RunnersUp (r:1 w:0)
 	// Storage: Elections Voting (r:1 w:1)
 	// Storage: Balances Locks (r:1 w:1)
+	/// The range of component `v` is `[1, 16]`.
 	fn vote_equal(v: u32, ) -> Weight {
-		(27_798_000 as Weight)
-			// Standard Error: 5_000
-			.saturating_add((238_000 as Weight).saturating_mul(v as Weight))
+		(26_143_000 as Weight)
+			// Standard Error: 23_000
+			.saturating_add((297_000 as Weight).saturating_mul(v as Weight))
 			.saturating_add(T::DbWeight::get().reads(5 as Weight))
 			.saturating_add(T::DbWeight::get().writes(2 as Weight))
 	}
@@ -79,10 +81,11 @@ impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> {
 	// Storage: Elections RunnersUp (r:1 w:0)
 	// Storage: Elections Voting (r:1 w:1)
 	// Storage: Balances Locks (r:1 w:1)
+	/// The range of component `v` is `[2, 16]`.
 	fn vote_more(v: u32, ) -> Weight {
-		(41_241_000 as Weight)
-			// Standard Error: 8_000
-			.saturating_add((259_000 as Weight).saturating_mul(v as Weight))
+		(40_431_000 as Weight)
+			// Standard Error: 6_000
+			.saturating_add((205_000 as Weight).saturating_mul(v as Weight))
 			.saturating_add(T::DbWeight::get().reads(5 as Weight))
 			.saturating_add(T::DbWeight::get().writes(2 as Weight))
 	}
@@ -91,35 +94,38 @@ impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> {
 	// Storage: Elections RunnersUp (r:1 w:0)
 	// Storage: Elections Voting (r:1 w:1)
 	// Storage: Balances Locks (r:1 w:1)
+	/// The range of component `v` is `[2, 16]`.
 	fn vote_less(v: u32, ) -> Weight {
-		(41_313_000 as Weight)
-			// Standard Error: 8_000
-			.saturating_add((255_000 as Weight).saturating_mul(v as Weight))
+		(40_188_000 as Weight)
+			// Standard Error: 6_000
+			.saturating_add((225_000 as Weight).saturating_mul(v as Weight))
 			.saturating_add(T::DbWeight::get().reads(5 as Weight))
 			.saturating_add(T::DbWeight::get().writes(2 as Weight))
 	}
 	// Storage: Elections Voting (r:1 w:1)
 	// Storage: Balances Locks (r:1 w:1)
 	fn remove_voter() -> Weight {
-		(39_218_000 as Weight)
+		(38_031_000 as Weight)
 			.saturating_add(T::DbWeight::get().reads(2 as Weight))
 			.saturating_add(T::DbWeight::get().writes(2 as Weight))
 	}
 	// Storage: Elections Candidates (r:1 w:1)
 	// Storage: Elections Members (r:1 w:0)
 	// Storage: Elections RunnersUp (r:1 w:0)
+	/// The range of component `c` is `[1, 1000]`.
 	fn submit_candidacy(c: u32, ) -> Weight {
-		(41_348_000 as Weight)
-			// Standard Error: 1_000
-			.saturating_add((112_000 as Weight).saturating_mul(c as Weight))
+		(43_715_000 as Weight)
+			// Standard Error: 0
+			.saturating_add((49_000 as Weight).saturating_mul(c as Weight))
 			.saturating_add(T::DbWeight::get().reads(3 as Weight))
 			.saturating_add(T::DbWeight::get().writes(1 as Weight))
 	}
 	// Storage: Elections Candidates (r:1 w:1)
+	/// The range of component `c` is `[1, 1000]`.
 	fn renounce_candidacy_candidate(c: u32, ) -> Weight {
-		(35_522_000 as Weight)
+		(47_882_000 as Weight)
 			// Standard Error: 1_000
-			.saturating_add((92_000 as Weight).saturating_mul(c as Weight))
+			.saturating_add((25_000 as Weight).saturating_mul(c as Weight))
 			.saturating_add(T::DbWeight::get().reads(1 as Weight))
 			.saturating_add(T::DbWeight::get().writes(1 as Weight))
 	}
@@ -129,13 +135,13 @@ impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> {
 	// Storage: Council Proposals (r:1 w:0)
 	// Storage: Council Members (r:0 w:1)
 	fn renounce_candidacy_members() -> Weight {
-		(47_887_000 as Weight)
+		(45_600_000 as Weight)
 			.saturating_add(T::DbWeight::get().reads(4 as Weight))
 			.saturating_add(T::DbWeight::get().writes(4 as Weight))
 	}
 	// Storage: Elections RunnersUp (r:1 w:1)
 	fn renounce_candidacy_runners_up() -> Weight {
-		(36_271_000 as Weight)
+		(34_959_000 as Weight)
 			.saturating_add(T::DbWeight::get().reads(1 as Weight))
 			.saturating_add(T::DbWeight::get().writes(1 as Weight))
 	}
@@ -143,34 +149,29 @@ impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> {
 	fn remove_member_without_replacement() -> Weight {
 		(2_000_000_000_000 as Weight)
 	}
-	// Storage: Elections RunnersUp (r:1 w:1)
 	// Storage: Elections Members (r:1 w:1)
 	// Storage: System Account (r:1 w:1)
+	// Storage: Elections RunnersUp (r:1 w:1)
 	// Storage: Council Prime (r:1 w:1)
 	// Storage: Council Proposals (r:1 w:0)
 	// Storage: Council Members (r:0 w:1)
 	fn remove_member_with_replacement() -> Weight {
-		(55_024_000 as Weight)
+		(52_684_000 as Weight)
 			.saturating_add(T::DbWeight::get().reads(5 as Weight))
 			.saturating_add(T::DbWeight::get().writes(5 as Weight))
 	}
-	// Storage: Elections RunnersUp (r:1 w:0)
-	fn remove_member_wrong_refund() -> Weight {
-		(13_089_000 as Weight)
-			.saturating_add(T::DbWeight::get().reads(1 as Weight))
-	}
-	// Storage: Elections Voting (r:251 w:250)
+	// Storage: Elections Voting (r:5001 w:5000)
 	// Storage: Elections Members (r:1 w:0)
 	// Storage: Elections RunnersUp (r:1 w:0)
 	// Storage: Elections Candidates (r:1 w:0)
-	// Storage: Balances Locks (r:250 w:250)
-	// Storage: System Account (r:250 w:250)
-	fn clean_defunct_voters(v: u32, d: u32, ) -> Weight {
+	// Storage: Balances Locks (r:5000 w:5000)
+	// Storage: System Account (r:5000 w:5000)
+	/// The range of component `v` is `[5000, 10000]`.
+	/// The range of component `d` is `[1, 5000]`.
+	fn clean_defunct_voters(v: u32, _d: u32, ) -> Weight {
 		(0 as Weight)
-			// Standard Error: 40_000
-			.saturating_add((51_848_000 as Weight).saturating_mul(v as Weight))
-			// Standard Error: 38_000
-			.saturating_add((537_000 as Weight).saturating_mul(d as Weight))
+			// Standard Error: 65_000
+			.saturating_add((64_009_000 as Weight).saturating_mul(v as Weight))
 			.saturating_add(T::DbWeight::get().reads(4 as Weight))
 			.saturating_add(T::DbWeight::get().reads((3 as Weight).saturating_mul(v as Weight)))
 			.saturating_add(T::DbWeight::get().writes((3 as Weight).saturating_mul(v as Weight)))
@@ -178,21 +179,23 @@ impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> {
 	// Storage: Elections Candidates (r:1 w:1)
 	// Storage: Elections Members (r:1 w:1)
 	// Storage: Elections RunnersUp (r:1 w:1)
-	// Storage: Elections Voting (r:502 w:0)
+	// Storage: Elections Voting (r:10001 w:0)
 	// Storage: Council Proposals (r:1 w:0)
 	// Storage: Elections ElectionRounds (r:1 w:1)
 	// Storage: Council Members (r:0 w:1)
 	// Storage: Council Prime (r:0 w:1)
-	// Storage: System Account (r:2 w:2)
+	// Storage: System Account (r:19 w:19)
+	/// The range of component `c` is `[1, 1000]`.
+	/// The range of component `v` is `[1, 10000]`.
+	/// The range of component `e` is `[10000, 160000]`.
 	fn election_phragmen(c: u32, v: u32, e: u32, ) -> Weight {
 		(0 as Weight)
-			// Standard Error: 1_664_000
-			.saturating_add((30_736_000 as Weight).saturating_mul(c as Weight))
-			// Standard Error: 692_000
-			.saturating_add((49_739_000 as Weight).saturating_mul(v as Weight))
-			// Standard Error: 47_000
-			.saturating_add((3_363_000 as Weight).saturating_mul(e as Weight))
-			.saturating_add(T::DbWeight::get().reads((2 as Weight).saturating_mul(c as Weight)))
+			// Standard Error: 778_000
+			.saturating_add((81_049_000 as Weight).saturating_mul(v as Weight))
+			// Standard Error: 51_000
+			.saturating_add((4_420_000 as Weight).saturating_mul(e as Weight))
+			.saturating_add(T::DbWeight::get().reads(279 as Weight))
+			.saturating_add(T::DbWeight::get().reads((1 as Weight).saturating_mul(c as Weight)))
 			.saturating_add(T::DbWeight::get().reads((1 as Weight).saturating_mul(v as Weight)))
 			.saturating_add(T::DbWeight::get().writes((1 as Weight).saturating_mul(c as Weight)))
 	}
@@ -205,10 +208,11 @@ impl WeightInfo for () {
 	// Storage: Elections RunnersUp (r:1 w:0)
 	// Storage: Elections Voting (r:1 w:1)
 	// Storage: Balances Locks (r:1 w:1)
+	/// The range of component `v` is `[1, 16]`.
 	fn vote_equal(v: u32, ) -> Weight {
-		(27_798_000 as Weight)
-			// Standard Error: 5_000
-			.saturating_add((238_000 as Weight).saturating_mul(v as Weight))
+		(26_143_000 as Weight)
+			// Standard Error: 23_000
+			.saturating_add((297_000 as Weight).saturating_mul(v as Weight))
 			.saturating_add(RocksDbWeight::get().reads(5 as Weight))
 			.saturating_add(RocksDbWeight::get().writes(2 as Weight))
 	}
@@ -217,10 +221,11 @@ impl WeightInfo for () {
 	// Storage: Elections RunnersUp (r:1 w:0)
 	// Storage: Elections Voting (r:1 w:1)
 	// Storage: Balances Locks (r:1 w:1)
+	/// The range of component `v` is `[2, 16]`.
 	fn vote_more(v: u32, ) -> Weight {
-		(41_241_000 as Weight)
-			// Standard Error: 8_000
-			.saturating_add((259_000 as Weight).saturating_mul(v as Weight))
+		(40_431_000 as Weight)
+			// Standard Error: 6_000
+			.saturating_add((205_000 as Weight).saturating_mul(v as Weight))
 			.saturating_add(RocksDbWeight::get().reads(5 as Weight))
 			.saturating_add(RocksDbWeight::get().writes(2 as Weight))
 	}
@@ -229,35 +234,38 @@ impl WeightInfo for () {
 	// Storage: Elections RunnersUp (r:1 w:0)
 	// Storage: Elections Voting (r:1 w:1)
 	// Storage: Balances Locks (r:1 w:1)
+	/// The range of component `v` is `[2, 16]`.
 	fn vote_less(v: u32, ) -> Weight {
-		(41_313_000 as Weight)
-			// Standard Error: 8_000
-			.saturating_add((255_000 as Weight).saturating_mul(v as Weight))
+		(40_188_000 as Weight)
+			// Standard Error: 6_000
+			.saturating_add((225_000 as Weight).saturating_mul(v as Weight))
 			.saturating_add(RocksDbWeight::get().reads(5 as Weight))
 			.saturating_add(RocksDbWeight::get().writes(2 as Weight))
 	}
 	// Storage: Elections Voting (r:1 w:1)
 	// Storage: Balances Locks (r:1 w:1)
 	fn remove_voter() -> Weight {
-		(39_218_000 as Weight)
+		(38_031_000 as Weight)
 			.saturating_add(RocksDbWeight::get().reads(2 as Weight))
 			.saturating_add(RocksDbWeight::get().writes(2 as Weight))
 	}
 	// Storage: Elections Candidates (r:1 w:1)
 	// Storage: Elections Members (r:1 w:0)
 	// Storage: Elections RunnersUp (r:1 w:0)
+	/// The range of component `c` is `[1, 1000]`.
 	fn submit_candidacy(c: u32, ) -> Weight {
-		(41_348_000 as Weight)
-			// Standard Error: 1_000
-			.saturating_add((112_000 as Weight).saturating_mul(c as Weight))
+		(43_715_000 as Weight)
+			// Standard Error: 0
+			.saturating_add((49_000 as Weight).saturating_mul(c as Weight))
 			.saturating_add(RocksDbWeight::get().reads(3 as Weight))
 			.saturating_add(RocksDbWeight::get().writes(1 as Weight))
 	}
 	// Storage: Elections Candidates (r:1 w:1)
+	/// The range of component `c` is `[1, 1000]`.
 	fn renounce_candidacy_candidate(c: u32, ) -> Weight {
-		(35_522_000 as Weight)
+		(47_882_000 as Weight)
 			// Standard Error: 1_000
-			.saturating_add((92_000 as Weight).saturating_mul(c as Weight))
+			.saturating_add((25_000 as Weight).saturating_mul(c as Weight))
 			.saturating_add(RocksDbWeight::get().reads(1 as Weight))
 			.saturating_add(RocksDbWeight::get().writes(1 as Weight))
 	}
@@ -267,13 +275,13 @@ impl WeightInfo for () {
 	// Storage: Council Proposals (r:1 w:0)
 	// Storage: Council Members (r:0 w:1)
 	fn renounce_candidacy_members() -> Weight {
-		(47_887_000 as Weight)
+		(45_600_000 as Weight)
 			.saturating_add(RocksDbWeight::get().reads(4 as Weight))
 			.saturating_add(RocksDbWeight::get().writes(4 as Weight))
 	}
 	// Storage: Elections RunnersUp (r:1 w:1)
 	fn renounce_candidacy_runners_up() -> Weight {
-		(36_271_000 as Weight)
+		(34_959_000 as Weight)
 			.saturating_add(RocksDbWeight::get().reads(1 as Weight))
 			.saturating_add(RocksDbWeight::get().writes(1 as Weight))
 	}
@@ -281,34 +289,29 @@ impl WeightInfo for () {
 	fn remove_member_without_replacement() -> Weight {
 		(2_000_000_000_000 as Weight)
 	}
-	// Storage: Elections RunnersUp (r:1 w:1)
 	// Storage: Elections Members (r:1 w:1)
 	// Storage: System Account (r:1 w:1)
+	// Storage: Elections RunnersUp (r:1 w:1)
 	// Storage: Council Prime (r:1 w:1)
 	// Storage: Council Proposals (r:1 w:0)
 	// Storage: Council Members (r:0 w:1)
 	fn remove_member_with_replacement() -> Weight {
-		(55_024_000 as Weight)
+		(52_684_000 as Weight)
 			.saturating_add(RocksDbWeight::get().reads(5 as Weight))
 			.saturating_add(RocksDbWeight::get().writes(5 as Weight))
 	}
-	// Storage: Elections RunnersUp (r:1 w:0)
-	fn remove_member_wrong_refund() -> Weight {
-		(13_089_000 as Weight)
-			.saturating_add(RocksDbWeight::get().reads(1 as Weight))
-	}
-	// Storage: Elections Voting (r:251 w:250)
+	// Storage: Elections Voting (r:5001 w:5000)
 	// Storage: Elections Members (r:1 w:0)
 	// Storage: Elections RunnersUp (r:1 w:0)
 	// Storage: Elections Candidates (r:1 w:0)
-	// Storage: Balances Locks (r:250 w:250)
-	// Storage: System Account (r:250 w:250)
-	fn clean_defunct_voters(v: u32, d: u32, ) -> Weight {
+	// Storage: Balances Locks (r:5000 w:5000)
+	// Storage: System Account (r:5000 w:5000)
+	/// The range of component `v` is `[5000, 10000]`.
+	/// The range of component `d` is `[1, 5000]`.
+	fn clean_defunct_voters(v: u32, _d: u32, ) -> Weight {
 		(0 as Weight)
-			// Standard Error: 40_000
-			.saturating_add((51_848_000 as Weight).saturating_mul(v as Weight))
-			// Standard Error: 38_000
-			.saturating_add((537_000 as Weight).saturating_mul(d as Weight))
+			// Standard Error: 65_000
+			.saturating_add((64_009_000 as Weight).saturating_mul(v as Weight))
 			.saturating_add(RocksDbWeight::get().reads(4 as Weight))
 			.saturating_add(RocksDbWeight::get().reads((3 as Weight).saturating_mul(v as Weight)))
 			.saturating_add(RocksDbWeight::get().writes((3 as Weight).saturating_mul(v as Weight)))
@@ -316,21 +319,23 @@ impl WeightInfo for () {
 	// Storage: Elections Candidates (r:1 w:1)
 	// Storage: Elections Members (r:1 w:1)
 	// Storage: Elections RunnersUp (r:1 w:1)
-	// Storage: Elections Voting (r:502 w:0)
+	// Storage: Elections Voting (r:10001 w:0)
 	// Storage: Council Proposals (r:1 w:0)
 	// Storage: Elections ElectionRounds (r:1 w:1)
 	// Storage: Council Members (r:0 w:1)
 	// Storage: Council Prime (r:0 w:1)
-	// Storage: System Account (r:2 w:2)
+	// Storage: System Account (r:19 w:19)
+	/// The range of component `c` is `[1, 1000]`.
+	/// The range of component `v` is `[1, 10000]`.
+	/// The range of component `e` is `[10000, 160000]`.
 	fn election_phragmen(c: u32, v: u32, e: u32, ) -> Weight {
 		(0 as Weight)
-			// Standard Error: 1_664_000
-			.saturating_add((30_736_000 as Weight).saturating_mul(c as Weight))
-			// Standard Error: 692_000
-			.saturating_add((49_739_000 as Weight).saturating_mul(v as Weight))
-			// Standard Error: 47_000
-			.saturating_add((3_363_000 as Weight).saturating_mul(e as Weight))
-			.saturating_add(RocksDbWeight::get().reads((2 as Weight).saturating_mul(c as Weight)))
+			// Standard Error: 778_000
+			.saturating_add((81_049_000 as Weight).saturating_mul(v as Weight))
+			// Standard Error: 51_000
+			.saturating_add((4_420_000 as Weight).saturating_mul(e as Weight))
+			.saturating_add(RocksDbWeight::get().reads(279 as Weight))
+			.saturating_add(RocksDbWeight::get().reads((1 as Weight).saturating_mul(c as Weight)))
 			.saturating_add(RocksDbWeight::get().reads((1 as Weight).saturating_mul(v as Weight)))
 			.saturating_add(RocksDbWeight::get().writes((1 as Weight).saturating_mul(c as Weight)))
 	}
-- 
GitLab