difftreelog
Merge pull request #810 from UniqueNetwork/fix/app-promotion-benchmarks
in: master
8 files changed
Cargo.lockdiffbeforeafterboth--- a/Cargo.lock
+++ b/Cargo.lock
@@ -5550,7 +5550,7 @@
[[package]]
name = "pallet-app-promotion"
-version = "0.1.2"
+version = "0.1.3"
dependencies = [
"frame-benchmarking",
"frame-support",
pallets/app-promotion/CHANGELOG.mddiffbeforeafterboth--- a/pallets/app-promotion/CHANGELOG.md
+++ b/pallets/app-promotion/CHANGELOG.md
@@ -4,6 +4,12 @@
<!-- bureaucrate goes here -->
+## [0.1.3] - 2022-12-25
+
+### Fixed
+
+- Benchmarks for `payoutStakers` and `unstake` extrinsics.
+
## [0.1.2] - 2022-12-20
### Fixed
pallets/app-promotion/Cargo.tomldiffbeforeafterboth--- a/pallets/app-promotion/Cargo.toml
+++ b/pallets/app-promotion/Cargo.toml
@@ -9,13 +9,13 @@
license = 'GPLv3'
name = 'pallet-app-promotion'
repository = 'https://github.com/UniqueNetwork/unique-chain'
-version = '0.1.2'
+version = '0.1.3'
[package.metadata.docs.rs]
targets = ['x86_64-unknown-linux-gnu']
[features]
-default = ['std']
+default = ['std',]
runtime-benchmarks = [
'frame-benchmarking',
'frame-support/runtime-benchmarks',
pallets/app-promotion/src/benchmarking.rsdiffbeforeafterboth77 } : _(RawOrigin::Root, T::CrossAccountId::from_sub(pallet_admin))77 } : _(RawOrigin::Root, T::CrossAccountId::from_sub(pallet_admin))787879 payout_stakers{79 payout_stakers{80 let b in 1..101;80 let b in 1..100;818182 let pallet_admin = account::<T::AccountId>("admin", 1, SEED);82 let pallet_admin = account::<T::AccountId>("admin", 1, SEED);83 let share = Perbill::from_rational(1u32, 20);83 let share = Perbill::from_rational(1u32, 20);84 PromototionPallet::<T>::set_admin_address(RawOrigin::Root.into(), T::CrossAccountId::from_sub(pallet_admin.clone()))?;84 PromototionPallet::<T>::set_admin_address(RawOrigin::Root.into(), T::CrossAccountId::from_sub(pallet_admin.clone()))?;85 <T as Config>::Currency::make_free_balance_be(&pallet_admin, Perbill::from_rational(1u32, 2) * BalanceOf::<T>::max_value());85 <T as Config>::Currency::make_free_balance_be(&pallet_admin, Perbill::from_rational(1u32, 2) * BalanceOf::<T>::max_value());8687 let staker: T::AccountId = account("caller", 0, SEED);88 <T as Config>::Currency::make_free_balance_be(&staker, Perbill::from_rational(1u32, 2) * BalanceOf::<T>::max_value());86 <T as Config>::Currency::make_free_balance_be(&<T as pallet::Config>::TreasuryAccountId::get(), Perbill::from_rational(1u32, 2) * BalanceOf::<T>::max_value());8789 let stakers: Vec<T::AccountId> = (0..b).map(|index| account("staker", index, SEED)).collect();88 let stakers: Vec<T::AccountId> = (0..b).map(|index| account("staker", index, SEED)).collect();90 stakers.iter().for_each(|staker| {89 stakers.iter().for_each(|staker| {91 <T as Config>::Currency::make_free_balance_be(&staker, Perbill::from_rational(1u32, 2) * BalanceOf::<T>::max_value());90 <T as Config>::Currency::make_free_balance_be(&staker, Perbill::from_rational(1u32, 2) * BalanceOf::<T>::max_value());92 });91 });93 (0..10).try_for_each(|_| {92 (1..11).try_for_each(|i| {93 <frame_system::Pallet<T>>::set_block_number(i.into());94 T::RelayBlockNumberProvider::set_block_number((2*i).into());95 assert_eq!(<frame_system::Pallet<T>>::block_number(), i.into());96 assert_eq!(T::RelayBlockNumberProvider::current_block_number(), (2*i).into());94 stakers.iter()97 stakers.iter()95 .map(|staker| {98 .map(|staker| {96 PromototionPallet::<T>::stake(RawOrigin::Signed(staker.clone()).into(), Into::<BalanceOf<T>>::into(100u128) * T::Nominal::get())99 PromototionPallet::<T>::stake(RawOrigin::Signed(staker.clone()).into(), Into::<BalanceOf<T>>::into(100u128) * T::Nominal::get())97 }).collect::<Result<Vec<_>, _>>()?;100 }).collect::<Result<Vec<_>, _>>()?;98 <frame_system::Pallet<T>>::finalize();10199 Result::<(), sp_runtime::DispatchError>::Ok(())102 Result::<(), sp_runtime::DispatchError>::Ok(())100 })?;103 })?;104105 let stakes = Staked::<T>::iter_prefix((&stakers[0],)).into_iter().collect::<Vec<_>>();106 assert_eq!(stakes.len(), 10);107108 <frame_system::Pallet<T>>::set_block_number(15_000.into());109 T::RelayBlockNumberProvider::set_block_number(30_000.into());101 } : _(RawOrigin::Signed(pallet_admin.clone()), Some(b as u8))110 } : _(RawOrigin::Signed(pallet_admin.clone()), Some(b as u8))102111103 stake {112 stake {110 let caller = account::<T::AccountId>("caller", 0, SEED);119 let caller = account::<T::AccountId>("caller", 0, SEED);111 let share = Perbill::from_rational(1u32, 20);120 let share = Perbill::from_rational(1u32, 20);112 let _ = <T as Config>::Currency::make_free_balance_be(&caller, Perbill::from_rational(1u32, 2) * BalanceOf::<T>::max_value());121 let _ = <T as Config>::Currency::make_free_balance_be(&caller, Perbill::from_rational(1u32, 2) * BalanceOf::<T>::max_value());113 (0..10).map(|_| {122 (1..11).map(|i| {114 // used to change block number123 // used to change block number115 <frame_system::Pallet<T>>::finalize();124 <frame_system::Pallet<T>>::set_block_number(i.into());125 T::RelayBlockNumberProvider::set_block_number((2*i).into());126 assert_eq!(<frame_system::Pallet<T>>::block_number(), i.into());127 assert_eq!(T::RelayBlockNumberProvider::current_block_number(), (2*i).into());116 PromototionPallet::<T>::stake(RawOrigin::Signed(caller.clone()).into(), share * <T as Config>::Currency::total_balance(&caller))128 PromototionPallet::<T>::stake(RawOrigin::Signed(caller.clone()).into(), share * <T as Config>::Currency::total_balance(&caller))117 }).collect::<Result<Vec<_>, _>>()?;129 }).collect::<Result<Vec<_>, _>>()?;118130pallets/app-promotion/src/weights.rsdiffbeforeafterboth--- a/pallets/app-promotion/src/weights.rs
+++ b/pallets/app-promotion/src/weights.rs
@@ -3,7 +3,7 @@
//! Autogenerated weights for pallet_app_promotion
//!
//! THIS FILE WAS AUTO-GENERATED USING THE SUBSTRATE BENCHMARK CLI VERSION 4.0.0-dev
-//! DATE: 2022-09-07, STEPS: `50`, REPEAT: 80, LOW RANGE: `[]`, HIGH RANGE: `[]`
+//! DATE: 2022-12-25, STEPS: `50`, REPEAT: 80, LOW RANGE: `[]`, HIGH RANGE: `[]`
//! EXECUTION: None, WASM-EXECUTION: Compiled, CHAIN: None, DB CACHE: 1024
// Executed Command:
@@ -51,77 +51,84 @@
// Storage: AppPromotion PendingUnstake (r:1 w:0)
// Storage: System Account (r:1 w:1)
fn on_initialize(b: u32, ) -> Weight {
- Weight::from_ref_time(2_651_000)
- // Standard Error: 103_000
- .saturating_add(Weight::from_ref_time(6_024_000).saturating_mul(b as u64))
+ Weight::from_ref_time(3_079_948 as u64)
+ // Standard Error: 30_376
+ .saturating_add(Weight::from_ref_time(6_343_630 as u64).saturating_mul(b as u64))
.saturating_add(T::DbWeight::get().reads(1 as u64))
.saturating_add(T::DbWeight::get().reads((1 as u64).saturating_mul(b as u64)))
.saturating_add(T::DbWeight::get().writes((1 as u64).saturating_mul(b as u64)))
}
// Storage: AppPromotion Admin (r:0 w:1)
fn set_admin_address() -> Weight {
- Weight::from_ref_time(7_117_000)
+ Weight::from_ref_time(6_653_000 as u64)
.saturating_add(T::DbWeight::get().writes(1 as u64))
}
// Storage: AppPromotion Admin (r:1 w:0)
+ // Storage: Configuration AppPromomotionConfigurationOverride (r:1 w:0)
// Storage: ParachainSystem ValidationData (r:1 w:0)
- // Storage: AppPromotion NextCalculatedRecord (r:1 w:1)
- // Storage: AppPromotion Staked (r:2 w:0)
+ // Storage: AppPromotion PreviousCalculatedRecord (r:1 w:1)
+ // Storage: AppPromotion Staked (r:11 w:10)
+ // Storage: System Account (r:2 w:2)
+ // Storage: Balances Locks (r:1 w:1)
+ // Storage: AppPromotion TotalStaked (r:1 w:1)
fn payout_stakers(b: u32, ) -> Weight {
- Weight::from_ref_time(9_958_000)
- // Standard Error: 8_000
- .saturating_add(Weight::from_ref_time(4_406_000).saturating_mul(b as u64))
- .saturating_add(T::DbWeight::get().reads(4 as u64))
- .saturating_add(T::DbWeight::get().reads((1 as u64).saturating_mul(b as u64)))
- .saturating_add(T::DbWeight::get().writes(1 as u64))
+ Weight::from_ref_time(74_048_000 as u64)
+ // Standard Error: 33_223
+ .saturating_add(Weight::from_ref_time(57_702_092 as u64).saturating_mul(b as u64))
+ .saturating_add(T::DbWeight::get().reads(7 as u64))
+ .saturating_add(T::DbWeight::get().reads((12 as u64).saturating_mul(b as u64)))
+ .saturating_add(T::DbWeight::get().writes(3 as u64))
+ .saturating_add(T::DbWeight::get().writes((12 as u64).saturating_mul(b as u64)))
}
// Storage: AppPromotion StakesPerAccount (r:1 w:1)
+ // Storage: Configuration AppPromomotionConfigurationOverride (r:1 w:0)
// Storage: System Account (r:1 w:1)
// Storage: Balances Locks (r:1 w:1)
// Storage: ParachainSystem ValidationData (r:1 w:0)
// Storage: AppPromotion Staked (r:1 w:1)
// Storage: AppPromotion TotalStaked (r:1 w:1)
fn stake() -> Weight {
- Weight::from_ref_time(20_574_000)
- .saturating_add(T::DbWeight::get().reads(6 as u64))
+ Weight::from_ref_time(20_314_000 as u64)
+ .saturating_add(T::DbWeight::get().reads(7 as u64))
.saturating_add(T::DbWeight::get().writes(5 as u64))
}
+ // Storage: Configuration AppPromomotionConfigurationOverride (r:1 w:0)
// Storage: AppPromotion PendingUnstake (r:1 w:1)
- // Storage: AppPromotion Staked (r:2 w:1)
+ // Storage: AppPromotion Staked (r:11 w:10)
// Storage: Balances Locks (r:1 w:1)
// Storage: System Account (r:1 w:1)
// Storage: AppPromotion TotalStaked (r:1 w:1)
// Storage: AppPromotion StakesPerAccount (r:0 w:1)
fn unstake() -> Weight {
- Weight::from_ref_time(31_703_000)
- .saturating_add(T::DbWeight::get().reads(6 as u64))
- .saturating_add(T::DbWeight::get().writes(6 as u64))
+ Weight::from_ref_time(64_582_000 as u64)
+ .saturating_add(T::DbWeight::get().reads(16 as u64))
+ .saturating_add(T::DbWeight::get().writes(15 as u64))
}
// Storage: AppPromotion Admin (r:1 w:0)
// Storage: Common CollectionById (r:1 w:1)
fn sponsor_collection() -> Weight {
- Weight::from_ref_time(12_932_000)
+ Weight::from_ref_time(16_364_000 as u64)
.saturating_add(T::DbWeight::get().reads(2 as u64))
.saturating_add(T::DbWeight::get().writes(1 as u64))
}
// Storage: AppPromotion Admin (r:1 w:0)
// Storage: Common CollectionById (r:1 w:1)
fn stop_sponsoring_collection() -> Weight {
- Weight::from_ref_time(12_453_000)
+ Weight::from_ref_time(15_710_000 as u64)
.saturating_add(T::DbWeight::get().reads(2 as u64))
.saturating_add(T::DbWeight::get().writes(1 as u64))
}
// Storage: AppPromotion Admin (r:1 w:0)
// Storage: EvmContractHelpers Sponsoring (r:0 w:1)
fn sponsor_contract() -> Weight {
- Weight::from_ref_time(11_952_000)
+ Weight::from_ref_time(12_669_000 as u64)
.saturating_add(T::DbWeight::get().reads(1 as u64))
.saturating_add(T::DbWeight::get().writes(1 as u64))
}
// Storage: AppPromotion Admin (r:1 w:0)
// Storage: EvmContractHelpers Sponsoring (r:1 w:1)
fn stop_sponsoring_contract() -> Weight {
- Weight::from_ref_time(12_538_000)
+ Weight::from_ref_time(14_406_000 as u64)
.saturating_add(T::DbWeight::get().reads(2 as u64))
.saturating_add(T::DbWeight::get().writes(1 as u64))
}
@@ -132,77 +139,84 @@
// Storage: AppPromotion PendingUnstake (r:1 w:0)
// Storage: System Account (r:1 w:1)
fn on_initialize(b: u32, ) -> Weight {
- Weight::from_ref_time(2_651_000)
- // Standard Error: 103_000
- .saturating_add(Weight::from_ref_time(6_024_000).saturating_mul(b as u64))
+ Weight::from_ref_time(3_079_948 as u64)
+ // Standard Error: 30_376
+ .saturating_add(Weight::from_ref_time(6_343_630 as u64).saturating_mul(b as u64))
.saturating_add(RocksDbWeight::get().reads(1 as u64))
.saturating_add(RocksDbWeight::get().reads((1 as u64).saturating_mul(b as u64)))
.saturating_add(RocksDbWeight::get().writes((1 as u64).saturating_mul(b as u64)))
}
// Storage: AppPromotion Admin (r:0 w:1)
fn set_admin_address() -> Weight {
- Weight::from_ref_time(7_117_000)
+ Weight::from_ref_time(6_653_000 as u64)
.saturating_add(RocksDbWeight::get().writes(1 as u64))
}
// Storage: AppPromotion Admin (r:1 w:0)
+ // Storage: Configuration AppPromomotionConfigurationOverride (r:1 w:0)
// Storage: ParachainSystem ValidationData (r:1 w:0)
- // Storage: AppPromotion NextCalculatedRecord (r:1 w:1)
- // Storage: AppPromotion Staked (r:2 w:0)
+ // Storage: AppPromotion PreviousCalculatedRecord (r:1 w:1)
+ // Storage: AppPromotion Staked (r:11 w:10)
+ // Storage: System Account (r:2 w:2)
+ // Storage: Balances Locks (r:1 w:1)
+ // Storage: AppPromotion TotalStaked (r:1 w:1)
fn payout_stakers(b: u32, ) -> Weight {
- Weight::from_ref_time(9_958_000)
- // Standard Error: 8_000
- .saturating_add(Weight::from_ref_time(4_406_000).saturating_mul(b as u64))
- .saturating_add(RocksDbWeight::get().reads(4 as u64))
- .saturating_add(RocksDbWeight::get().reads((1 as u64).saturating_mul(b as u64)))
- .saturating_add(RocksDbWeight::get().writes(1 as u64))
+ Weight::from_ref_time(74_048_000 as u64)
+ // Standard Error: 33_223
+ .saturating_add(Weight::from_ref_time(57_702_092 as u64).saturating_mul(b as u64))
+ .saturating_add(RocksDbWeight::get().reads(7 as u64))
+ .saturating_add(RocksDbWeight::get().reads((12 as u64).saturating_mul(b as u64)))
+ .saturating_add(RocksDbWeight::get().writes(3 as u64))
+ .saturating_add(RocksDbWeight::get().writes((12 as u64).saturating_mul(b as u64)))
}
// Storage: AppPromotion StakesPerAccount (r:1 w:1)
+ // Storage: Configuration AppPromomotionConfigurationOverride (r:1 w:0)
// Storage: System Account (r:1 w:1)
// Storage: Balances Locks (r:1 w:1)
// Storage: ParachainSystem ValidationData (r:1 w:0)
// Storage: AppPromotion Staked (r:1 w:1)
// Storage: AppPromotion TotalStaked (r:1 w:1)
fn stake() -> Weight {
- Weight::from_ref_time(20_574_000)
- .saturating_add(RocksDbWeight::get().reads(6 as u64))
+ Weight::from_ref_time(20_314_000 as u64)
+ .saturating_add(RocksDbWeight::get().reads(7 as u64))
.saturating_add(RocksDbWeight::get().writes(5 as u64))
}
+ // Storage: Configuration AppPromomotionConfigurationOverride (r:1 w:0)
// Storage: AppPromotion PendingUnstake (r:1 w:1)
- // Storage: AppPromotion Staked (r:2 w:1)
+ // Storage: AppPromotion Staked (r:11 w:10)
// Storage: Balances Locks (r:1 w:1)
// Storage: System Account (r:1 w:1)
// Storage: AppPromotion TotalStaked (r:1 w:1)
// Storage: AppPromotion StakesPerAccount (r:0 w:1)
fn unstake() -> Weight {
- Weight::from_ref_time(31_703_000)
- .saturating_add(RocksDbWeight::get().reads(6 as u64))
- .saturating_add(RocksDbWeight::get().writes(6 as u64))
+ Weight::from_ref_time(64_582_000 as u64)
+ .saturating_add(RocksDbWeight::get().reads(16 as u64))
+ .saturating_add(RocksDbWeight::get().writes(15 as u64))
}
// Storage: AppPromotion Admin (r:1 w:0)
// Storage: Common CollectionById (r:1 w:1)
fn sponsor_collection() -> Weight {
- Weight::from_ref_time(12_932_000)
+ Weight::from_ref_time(16_364_000 as u64)
.saturating_add(RocksDbWeight::get().reads(2 as u64))
.saturating_add(RocksDbWeight::get().writes(1 as u64))
}
// Storage: AppPromotion Admin (r:1 w:0)
// Storage: Common CollectionById (r:1 w:1)
fn stop_sponsoring_collection() -> Weight {
- Weight::from_ref_time(12_453_000)
+ Weight::from_ref_time(15_710_000 as u64)
.saturating_add(RocksDbWeight::get().reads(2 as u64))
.saturating_add(RocksDbWeight::get().writes(1 as u64))
}
// Storage: AppPromotion Admin (r:1 w:0)
// Storage: EvmContractHelpers Sponsoring (r:0 w:1)
fn sponsor_contract() -> Weight {
- Weight::from_ref_time(11_952_000)
+ Weight::from_ref_time(12_669_000 as u64)
.saturating_add(RocksDbWeight::get().reads(1 as u64))
.saturating_add(RocksDbWeight::get().writes(1 as u64))
}
// Storage: AppPromotion Admin (r:1 w:0)
// Storage: EvmContractHelpers Sponsoring (r:1 w:1)
fn stop_sponsoring_contract() -> Weight {
- Weight::from_ref_time(12_538_000)
+ Weight::from_ref_time(14_406_000 as u64)
.saturating_add(RocksDbWeight::get().reads(2 as u64))
.saturating_add(RocksDbWeight::get().writes(1 as u64))
}
runtime/common/mod.rsdiffbeforeafterboth--- a/runtime/common/mod.rs
+++ b/runtime/common/mod.rs
@@ -145,6 +145,10 @@
.map(|d| d.relay_parent_number)
.unwrap_or_default()
}
+ #[cfg(feature = "runtime-benchmarks")]
+ fn set_block_number(block: Self::BlockNumber) {
+ cumulus_pallet_parachain_system::RelaychainBlockNumberProvider::<T>::set_block_number(block)
+ }
}
pub(crate) struct CheckInherents;
runtime/opal/Cargo.tomldiffbeforeafterboth--- a/runtime/opal/Cargo.toml
+++ b/runtime/opal/Cargo.toml
@@ -44,6 +44,7 @@
'sp-runtime/runtime-benchmarks',
'xcm-builder/runtime-benchmarks',
'pallet-maintenance/runtime-benchmarks',
+ 'cumulus-pallet-parachain-system/runtime-benchmarks'
]
try-runtime = [
'frame-try-runtime',
runtime/quartz/Cargo.tomldiffbeforeafterboth--- a/runtime/quartz/Cargo.toml
+++ b/runtime/quartz/Cargo.toml
@@ -43,6 +43,7 @@
'sp-runtime/runtime-benchmarks',
'xcm-builder/runtime-benchmarks',
'pallet-maintenance/runtime-benchmarks',
+ 'cumulus-pallet-parachain-system/runtime-benchmarks',
]
try-runtime = [
'frame-try-runtime',