Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Collective: Benchmark with greater MaxProposals #12454

Merged
paritytech-processbot merged 19 commits into from
Nov 14, 2022
Merged

Collective: Benchmark with greater MaxProposals #12454

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
a8bc2cc
Collective: Benchmark with greated
Szegoo Oct 9, 2022
8ce5fa2
fix
Szegoo Oct 9, 2022
ddefb59
remove bs
Szegoo Oct 9, 2022
d3e2579
id_to_remark_data
Szegoo Oct 9, 2022
0830040
fix
Szegoo Oct 10, 2022
08fcd43
remove hardcoded
Szegoo Oct 10, 2022
a887818
clean up
Szegoo Oct 11, 2022
375f999
simplify
Szegoo Oct 11, 2022
93586a2
questionable renaming
Szegoo Oct 11, 2022
167efdb
better variable name
Szegoo Oct 11, 2022
80e9937
better solution
Szegoo Oct 11, 2022
bb65d9a
no need for large length
Szegoo Oct 11, 2022
61deeac
better solution
Szegoo Oct 12, 2022
79416e6
Update frame/collective/src/benchmarking.rs
Szegoo Oct 13, 2022
f57994a
fix
Szegoo Oct 13, 2022
1f5208d
test
Szegoo Oct 13, 2022
1769703
remove test
Szegoo Oct 13, 2022
bd89f9b
Merge branch 'paritytech:master' into collective-bench
Szegoo Oct 13, 2022
6e2de72
Merge branch 'paritytech:master' into collective-bench
Szegoo Nov 13, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
id_to_remark_data
  • Loading branch information
@Szegoo
Szegoo committed Oct 9, 2022
commit d3e25799c9e19a4c5edfd6983254b254a1ef3eb5
18 changes: 10 additions & 8 deletions frame/collective/src/benchmarking.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,10 @@ fn assert_last_event<T: Config<I>, I: 'static>(generic_event: <T as Config<I>>::
frame_system::Pallet::<T>::assert_last_event(generic_event.into());
}

fn id_to_remark_data<T: Config<I>, I: 'static>(id: u32, length: usize) -> SystemCall<T> {
SystemCall::<T>::remark { remark: vec![id.try_into().unwrap(); length] }
}

benchmarks_instance_pallet! {
set_members {
let m in 0 .. T::MaxMembers::get();
Expand Down Expand Up @@ -64,9 +68,7 @@ benchmarks_instance_pallet! {
let length = 100;
for i in 0 .. p {
// Proposals should be different so that different proposal hashes are generated
let proposal: T::Proposal = SystemCall::<T>::remark_vec_u32 {
remark: vec![i; length]
}.into();
let proposal: T::Proposal = id_to_remark_data(i, length).into();
Collective::<T, I>::propose(
SystemOrigin::Signed(old_members.last().unwrap().clone()).into(),
threshold,
Expand Down Expand Up @@ -186,7 +188,7 @@ benchmarks_instance_pallet! {
// Add previous proposals.
for i in 0 .. p - 1 {
// Proposals should be different so that different proposal hashes are generated
let proposal: T::Proposal = SystemCall::<T>::remark_vec_u32 { remark: vec![i; b as usize] }.into();
let proposal: T::Proposal = id_to_remark_data(i, b as usize).into();
Collective::<T, I>::propose(
SystemOrigin::Signed(caller.clone()).into(),
threshold,
Expand Down Expand Up @@ -234,7 +236,7 @@ benchmarks_instance_pallet! {
let mut last_hash = T::Hash::default();
for i in 0 .. p {
// Proposals should be different so that different proposal hashes are generated
let proposal: T::Proposal = SystemCall::<T>::remark_vec_u32 { remark: vec![i; b as usize] }.into();
let proposal: T::Proposal = id_to_remark_data(i, b as usize).into();
Collective::<T, I>::propose(
SystemOrigin::Signed(proposer.clone()).into(),
threshold,
Expand Down Expand Up @@ -388,7 +390,7 @@ benchmarks_instance_pallet! {
let mut last_hash = T::Hash::default();
for i in 0 .. p {
// Proposals should be different so that different proposal hashes are generated
let proposal: T::Proposal = SystemCall::<T>::remark_vec_u32 { remark: vec![i; b as usize] }.into();
let proposal: T::Proposal = id_to_remark_data(i, b as usize).into();
Collective::<T, I>::propose(
SystemOrigin::Signed(caller.clone()).into(),
threshold,
Expand Down Expand Up @@ -548,7 +550,7 @@ benchmarks_instance_pallet! {
let mut last_hash = T::Hash::default();
for i in 0 .. p {
// Proposals should be different so that different proposal hashes are generated
let proposal: T::Proposal = SystemCall::<T>::remark_vec_u32 { remark: vec![i; b as usize] }.into();
let proposal: T::Proposal = id_to_remark_data(i, b as usize).into();
Collective::<T, I>::propose(
SystemOrigin::Signed(caller.clone()).into(),
threshold,
Expand Down Expand Up @@ -619,7 +621,7 @@ benchmarks_instance_pallet! {
let mut last_hash = T::Hash::default();
for i in 0 .. p {
// Proposals should be different so that different proposal hashes are generated
let proposal: T::Proposal = SystemCall::<T>::remark_vec_u32 { remark: vec![i; b as usize] }.into();
let proposal: T::Proposal = id_to_remark_data(i, b as usize).into();
Collective::<T, I>::propose(
SystemOrigin::Signed(caller.clone()).into(),
threshold,
Expand Down