change: Fix, recalculate and use bridge pallet WeightInfo (#992)

Author: AmbientTea

toolkit/bridge/pallet/src/benchmarking.rs

@@ -34,10 +34,7 @@ mod benchmarks {
 	use super::*;
 
 	#[benchmark]
-	fn handle_transfers(
-		t: Linear<1, { T::MaxTransfersPerBlock::get() }>,
-		s: Linear<1, { T::MaxTransfersPerBlock::get() }>,
-	) {
+	fn handle_transfers(t: Linear<1, { T::MaxTransfersPerBlock::get() }>) {
 		assert_ok!(Pallet::<T>::set_main_chain_scripts(
 			RawOrigin::Root.into(),
 			T::BenchmarkHelper::main_chain_scripts(),

toolkit/bridge/pallet/src/lib.rs

@@ -39,6 +39,7 @@ impl<Recipient> TransferHandler<Recipient> for () {
 #[frame_support::pallet(dev_mode)]
 pub mod pallet {
 	use super::*;
+	use crate::weights::WeightInfo;
 	use frame_support::pallet_prelude::*;
 	use frame_system::{ensure_none, pallet_prelude::OriginFor};
 	use parity_scale_codec::MaxEncodedLen;
@@ -112,7 +113,7 @@ pub mod pallet {
 	impl<T: Config> Pallet<T> {
 		/// Inherent extrinsic that handles all incoming transfers in the current block
 		#[pallet::call_index(0)]
-		#[pallet::weight(0)]
+		#[pallet::weight(T::WeightInfo::handle_transfers(transfers.len() as u32))]
 		pub fn handle_transfers(
 			origin: OriginFor<T>,
 			transfers: BoundedVec<BridgeTransferV1<T::Recipient>, T::MaxTransfersPerBlock>,
@@ -132,7 +133,7 @@ pub mod pallet {
 		///
 		///
 		#[pallet::call_index(1)]
-		#[pallet::weight(0)]
+		#[pallet::weight(T::WeightInfo::set_main_chain_scripts())]
 		pub fn set_main_chain_scripts(
 			origin: OriginFor<T>,
 			new_scripts: MainChainScripts,

toolkit/bridge/pallet/src/weights.rs

@@ -2,7 +2,7 @@
 //! Autogenerated weights for pallet_partner_chains_bridge
 //!
 //! THIS FILE WAS AUTO-GENERATED USING THE SUBSTRATE BENCHMARK CLI VERSION 50.0.0
-//! DATE: 2025-08-25, STEPS: `500`, REPEAT: `200`, LOW RANGE: `[]`, HIGH RANGE: `[]`
+//! DATE: 2025-09-10, STEPS: `500`, REPEAT: `200`, LOW RANGE: `[]`, HIGH RANGE: `[]`
 //! WORST CASE MAP SIZE: `1000000`
 //! HOSTNAME: `Nikolaoss-MacBook-Pro.local`, CPU: `<UNKNOWN>`
 //! EXECUTION: , WASM-EXECUTION: Compiled, CHAIN: None, DB CACHE: 1024
@@ -36,61 +36,65 @@ use core::marker::PhantomData;
 
 /// Weight functions needed for pallet_partner_chains_bridge.
 pub trait WeightInfo {
-	fn handle_transfers(t: u32, s: u32, ) -> Weight;
+	fn handle_transfers(t: u32, ) -> Weight;
 	fn set_main_chain_scripts() -> Weight;
 }
 
 /// Weights for pallet_partner_chains_bridge using the Substrate node and recommended hardware.
 pub struct SubstrateWeight<T>(PhantomData<T>);
 impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> {
+	/// Storage: `Bridge::DataCheckpoint` (r:0 w:1)
+	/// Proof: `Bridge::DataCheckpoint` (`max_values`: Some(1), `max_size`: None, mode: `Measured`)
 	/// The range of component `t` is `[1, 256]`.
-	/// The range of component `s` is `[1, 256]`.
-	fn handle_transfers(t: u32, s: u32, ) -> Weight {
+	fn handle_transfers(t: u32, ) -> Weight {
 		// Proof Size summary in bytes:
 		//  Measured:  `0`
 		//  Estimated: `0`
-		// Minimum execution time: 1_000_000 picoseconds.
-		Weight::from_parts(1_914_402, 0)
-			// Standard Error: 8
-			.saturating_add(Weight::from_parts(166, 0).saturating_mul(t.into()))
-			// Standard Error: 8
-			.saturating_add(Weight::from_parts(127, 0).saturating_mul(s.into()))
+		// Minimum execution time: 2_000_000 picoseconds.
+		Weight::from_parts(1_999_689, 0)
+			// Standard Error: 4
+			.saturating_add(Weight::from_parts(53, 0).saturating_mul(t.into()))
+			.saturating_add(T::DbWeight::get().writes(1_u64))
 	}
 	/// Storage: `Bridge::MainChainScriptsConfiguration` (r:0 w:1)
 	/// Proof: `Bridge::MainChainScriptsConfiguration` (`max_values`: Some(1), `max_size`: None, mode: `Measured`)
+	/// Storage: `Bridge::DataCheckpoint` (r:0 w:1)
+	/// Proof: `Bridge::DataCheckpoint` (`max_values`: Some(1), `max_size`: None, mode: `Measured`)
 	fn set_main_chain_scripts() -> Weight {
 		// Proof Size summary in bytes:
 		//  Measured:  `0`
 		//  Estimated: `0`
-		// Minimum execution time: 1_000_000 picoseconds.
+		// Minimum execution time: 2_000_000 picoseconds.
 		Weight::from_parts(2_000_000, 0)
-			.saturating_add(T::DbWeight::get().writes(1_u64))
+			.saturating_add(T::DbWeight::get().writes(2_u64))
 	}
 }
 
 // For backwards compatibility and tests
 impl WeightInfo for () {
+	/// Storage: `Bridge::DataCheckpoint` (r:0 w:1)
+	/// Proof: `Bridge::DataCheckpoint` (`max_values`: Some(1), `max_size`: None, mode: `Measured`)
 	/// The range of component `t` is `[1, 256]`.
-	/// The range of component `s` is `[1, 256]`.
-	fn handle_transfers(t: u32, s: u32, ) -> Weight {
+	fn handle_transfers(t: u32, ) -> Weight {
 		// Proof Size summary in bytes:
 		//  Measured:  `0`
 		//  Estimated: `0`
-		// Minimum execution time: 1_000_000 picoseconds.
-		Weight::from_parts(1_914_402, 0)
-			// Standard Error: 8
-			.saturating_add(Weight::from_parts(166, 0).saturating_mul(t.into()))
-			// Standard Error: 8
-			.saturating_add(Weight::from_parts(127, 0).saturating_mul(s.into()))
+		// Minimum execution time: 2_000_000 picoseconds.
+		Weight::from_parts(1_999_689, 0)
+			// Standard Error: 4
+			.saturating_add(Weight::from_parts(53, 0).saturating_mul(t.into()))
+			.saturating_add(RocksDbWeight::get().writes(1_u64))
 	}
 	/// Storage: `Bridge::MainChainScriptsConfiguration` (r:0 w:1)
 	/// Proof: `Bridge::MainChainScriptsConfiguration` (`max_values`: Some(1), `max_size`: None, mode: `Measured`)
+	/// Storage: `Bridge::DataCheckpoint` (r:0 w:1)
+	/// Proof: `Bridge::DataCheckpoint` (`max_values`: Some(1), `max_size`: None, mode: `Measured`)
 	fn set_main_chain_scripts() -> Weight {
 		// Proof Size summary in bytes:
 		//  Measured:  `0`
 		//  Estimated: `0`
-		// Minimum execution time: 1_000_000 picoseconds.
+		// Minimum execution time: 2_000_000 picoseconds.
 		Weight::from_parts(2_000_000, 0)
-			.saturating_add(RocksDbWeight::get().writes(1_u64))
+			.saturating_add(RocksDbWeight::get().writes(2_u64))
 	}
 }