This repository was archived by the owner on Nov 15, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Use the automated call decode weight for submit_unsigned benchmark
#9766
Merged
+94
−100
Merged
Changes from 5 commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
e91b5df
Use the automated call decode weight for benchmark
kianenigma d9ffcd5
Merge branch 'master' of https://github.com/paritytech/substrate into…
d041fe8
cargo run --quiet --release --features=runtime-benchmarks --manifest-…
d32a30a
Master.into()
kianenigma 2ac99ba
Merge branch 'kiz-fix-decode-weight-election' of github.com:paritytec…
kianenigma d555b46
Merge branch 'master' of https://github.com/paritytech/substrate into…
0aaf4bb
cargo run --quiet --release --features=runtime-benchmarks --manifest-…
e7507ce
Merge branch 'master' of https://github.com/paritytech/substrate into…
8b44cb3
cargo run --quiet --release --features=runtime-benchmarks --manifest-…
af312e4
cargo run --quiet --release --features=runtime-benchmarks --manifest-…
0acb048
reduce the boundaries
kianenigma bce1de6
Fix
kianenigma cf3edf7
Fox
kianenigma df9f0ea
cargo run --quiet --release --features=runtime-benchmarks --manifest-…
bd1011f
reduce the numbers even less
kianenigma 19dd065
Merged
kianenigma 62af610
Merge branch 'master' of https://github.com/paritytech/substrate into…
0e444a5
cargo run --quiet --release --features=runtime-benchmarks --manifest-…
602d8b8
Fix a few things
kianenigma 836fe89
Merge branch 'kiz-fix-decode-weight-election' of github.com:paritytec…
kianenigma fd63969
Merge branch 'master' of https://github.com/paritytech/substrate into…
d9b8e23
cargo run --quiet --release --features=runtime-benchmarks --manifest-…
8d8f6c3
Merge branch 'master' of https://github.com/paritytech/substrate into…
33a94d7
cargo run --quiet --release --features=runtime-benchmarks --manifest-…
407cbc1
update comment
kianenigma be355e3
Merge branch 'kiz-fix-decode-weight-election' of github.com:paritytec…
kianenigma File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -18,7 +18,7 @@ | |
| //! Autogenerated weights for pallet_election_provider_multi_phase | ||
| //! | ||
| //! THIS FILE WAS AUTO-GENERATED USING THE SUBSTRATE BENCHMARK CLI VERSION 4.0.0-dev | ||
| //! DATE: 2021-09-21, STEPS: `50`, REPEAT: 20, LOW RANGE: `[]`, HIGH RANGE: `[]` | ||
| //! DATE: 2021-09-22, STEPS: `50`, REPEAT: 20, LOW RANGE: `[]`, HIGH RANGE: `[]` | ||
| //! EXECUTION: Some(Wasm), WASM-EXECUTION: Compiled, CHAIN: Some("dev"), DB CACHE: 128 | ||
|
|
||
| // Executed Command: | ||
|
|
@@ -69,45 +69,45 @@ impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> { | |
| // Storage: Staking ForceEra (r:1 w:0) | ||
| // Storage: ElectionProviderMultiPhase CurrentPhase (r:1 w:0) | ||
| fn on_initialize_nothing() -> Weight { | ||
| (22_696_000 as Weight) | ||
| (22_784_000 as Weight) | ||
| .saturating_add(T::DbWeight::get().reads(8 as Weight)) | ||
| } | ||
| // Storage: ElectionProviderMultiPhase Round (r:1 w:0) | ||
| // Storage: ElectionProviderMultiPhase CurrentPhase (r:0 w:1) | ||
| fn on_initialize_open_signed() -> Weight { | ||
| (30_317_000 as Weight) | ||
| (32_763_000 as Weight) | ||
| .saturating_add(T::DbWeight::get().reads(1 as Weight)) | ||
| .saturating_add(T::DbWeight::get().writes(1 as Weight)) | ||
| } | ||
| // Storage: ElectionProviderMultiPhase Round (r:1 w:0) | ||
| // Storage: ElectionProviderMultiPhase CurrentPhase (r:0 w:1) | ||
| fn on_initialize_open_unsigned() -> Weight { | ||
| (30_661_000 as Weight) | ||
| (29_117_000 as Weight) | ||
| .saturating_add(T::DbWeight::get().reads(1 as Weight)) | ||
| .saturating_add(T::DbWeight::get().writes(1 as Weight)) | ||
| } | ||
| // Storage: System Account (r:1 w:1) | ||
| // Storage: ElectionProviderMultiPhase QueuedSolution (r:0 w:1) | ||
| fn finalize_signed_phase_accept_solution() -> Weight { | ||
| (48_439_000 as Weight) | ||
| (48_996_000 as Weight) | ||
| .saturating_add(T::DbWeight::get().reads(1 as Weight)) | ||
| .saturating_add(T::DbWeight::get().writes(2 as Weight)) | ||
| } | ||
| // Storage: System Account (r:1 w:1) | ||
| fn finalize_signed_phase_reject_solution() -> Weight { | ||
| (32_072_000 as Weight) | ||
| (32_508_000 as Weight) | ||
| .saturating_add(T::DbWeight::get().reads(1 as Weight)) | ||
| .saturating_add(T::DbWeight::get().writes(1 as Weight)) | ||
| } | ||
| // Storage: ElectionProviderMultiPhase SnapshotMetadata (r:0 w:1) | ||
| // Storage: ElectionProviderMultiPhase DesiredTargets (r:0 w:1) | ||
| // Storage: ElectionProviderMultiPhase Snapshot (r:0 w:1) | ||
| fn create_snapshot_internal(v: u32, t: u32, ) -> Weight { | ||
| (89_239_000 as Weight) | ||
| (96_001_000 as Weight) | ||
| // Standard Error: 1_000 | ||
| .saturating_add((320_000 as Weight).saturating_mul(v as Weight)) | ||
| .saturating_add((307_000 as Weight).saturating_mul(v as Weight)) | ||
| // Standard Error: 2_000 | ||
| .saturating_add((129_000 as Weight).saturating_mul(t as Weight)) | ||
| .saturating_add((133_000 as Weight).saturating_mul(t as Weight)) | ||
| .saturating_add(T::DbWeight::get().writes(3 as Weight)) | ||
| } | ||
| // Storage: ElectionProviderMultiPhase SignedSubmissionIndices (r:1 w:1) | ||
|
|
@@ -120,11 +120,11 @@ impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> { | |
| // Storage: ElectionProviderMultiPhase Snapshot (r:0 w:1) | ||
| // Storage: ElectionProviderMultiPhase CurrentPhase (r:0 w:1) | ||
| fn elect_queued(a: u32, d: u32, ) -> Weight { | ||
| (156_010_000 as Weight) | ||
| (100_505_000 as Weight) | ||
| // Standard Error: 6_000 | ||
| .saturating_add((1_579_000 as Weight).saturating_mul(a as Weight)) | ||
| // Standard Error: 9_000 | ||
| .saturating_add((299_000 as Weight).saturating_mul(d as Weight)) | ||
| .saturating_add((1_665_000 as Weight).saturating_mul(a as Weight)) | ||
| // Standard Error: 10_000 | ||
| .saturating_add((443_000 as Weight).saturating_mul(d as Weight)) | ||
| .saturating_add(T::DbWeight::get().reads(6 as Weight)) | ||
| .saturating_add(T::DbWeight::get().writes(8 as Weight)) | ||
| } | ||
|
|
@@ -135,9 +135,9 @@ impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> { | |
| // Storage: ElectionProviderMultiPhase SignedSubmissionNextIndex (r:1 w:1) | ||
| // Storage: ElectionProviderMultiPhase SignedSubmissionsMap (r:0 w:1) | ||
| fn submit(c: u32, ) -> Weight { | ||
| (73_498_000 as Weight) | ||
| // Standard Error: 73_000 | ||
| .saturating_add((222_000 as Weight).saturating_mul(c as Weight)) | ||
| (74_088_000 as Weight) | ||
| // Standard Error: 59_000 | ||
| .saturating_add((187_000 as Weight).saturating_mul(c as Weight)) | ||
| .saturating_add(T::DbWeight::get().reads(5 as Weight)) | ||
| .saturating_add(T::DbWeight::get().writes(3 as Weight)) | ||
| } | ||
|
|
@@ -151,13 +151,13 @@ impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> { | |
| fn submit_unsigned(v: u32, t: u32, a: u32, d: u32, ) -> Weight { | ||
| (0 as Weight) | ||
| // Standard Error: 5_000 | ||
| .saturating_add((2_028_000 as Weight).saturating_mul(v as Weight)) | ||
| // Standard Error: 11_000 | ||
| .saturating_add((124_000 as Weight).saturating_mul(t as Weight)) | ||
| .saturating_add((1_970_000 as Weight).saturating_mul(v as Weight)) | ||
| // Standard Error: 10_000 | ||
| .saturating_add((173_000 as Weight).saturating_mul(t as Weight)) | ||
| // Standard Error: 18_000 | ||
| .saturating_add((9_991_000 as Weight).saturating_mul(a as Weight)) | ||
| // Standard Error: 28_000 | ||
| .saturating_add((2_365_000 as Weight).saturating_mul(d as Weight)) | ||
| .saturating_add((9_783_000 as Weight).saturating_mul(a as Weight)) | ||
| // Standard Error: 27_000 | ||
| .saturating_add((2_224_000 as Weight).saturating_mul(d as Weight)) | ||
| .saturating_add(T::DbWeight::get().reads(7 as Weight)) | ||
| .saturating_add(T::DbWeight::get().writes(1 as Weight)) | ||
| } | ||
|
|
@@ -168,13 +168,13 @@ impl<T: frame_system::Config> WeightInfo for SubstrateWeight<T> { | |
| fn feasibility_check(v: u32, t: u32, a: u32, d: u32, ) -> Weight { | ||
| (0 as Weight) | ||
| // Standard Error: 3_000 | ||
| .saturating_add((1_935_000 as Weight).saturating_mul(v as Weight)) | ||
| // Standard Error: 6_000 | ||
| .saturating_add((79_000 as Weight).saturating_mul(t as Weight)) | ||
| // Standard Error: 10_000 | ||
| .saturating_add((7_904_000 as Weight).saturating_mul(a as Weight)) | ||
| // Standard Error: 16_000 | ||
| .saturating_add((1_807_000 as Weight).saturating_mul(d as Weight)) | ||
| .saturating_add((1_910_000 as Weight).saturating_mul(v as Weight)) | ||
| // Standard Error: 7_000 | ||
| .saturating_add((111_000 as Weight).saturating_mul(t as Weight)) | ||
| // Standard Error: 13_000 | ||
| .saturating_add((7_741_000 as Weight).saturating_mul(a as Weight)) | ||
| // Standard Error: 19_000 | ||
| .saturating_add((1_844_000 as Weight).saturating_mul(d as Weight)) | ||
| .saturating_add(T::DbWeight::get().reads(4 as Weight)) | ||
| } | ||
| } | ||
|
|
@@ -190,45 +190,45 @@ impl WeightInfo for () { | |
| // Storage: Staking ForceEra (r:1 w:0) | ||
| // Storage: ElectionProviderMultiPhase CurrentPhase (r:1 w:0) | ||
| fn on_initialize_nothing() -> Weight { | ||
| (22_696_000 as Weight) | ||
| (22_784_000 as Weight) | ||
| .saturating_add(RocksDbWeight::get().reads(8 as Weight)) | ||
| } | ||
| // Storage: ElectionProviderMultiPhase Round (r:1 w:0) | ||
| // Storage: ElectionProviderMultiPhase CurrentPhase (r:0 w:1) | ||
| fn on_initialize_open_signed() -> Weight { | ||
| (30_317_000 as Weight) | ||
| (32_763_000 as Weight) | ||
| .saturating_add(RocksDbWeight::get().reads(1 as Weight)) | ||
| .saturating_add(RocksDbWeight::get().writes(1 as Weight)) | ||
| } | ||
| // Storage: ElectionProviderMultiPhase Round (r:1 w:0) | ||
| // Storage: ElectionProviderMultiPhase CurrentPhase (r:0 w:1) | ||
| fn on_initialize_open_unsigned() -> Weight { | ||
| (30_661_000 as Weight) | ||
| (29_117_000 as Weight) | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. seriously, this function is super trivial. I am not sure why it is changing by like 20%. |
||
| .saturating_add(RocksDbWeight::get().reads(1 as Weight)) | ||
| .saturating_add(RocksDbWeight::get().writes(1 as Weight)) | ||
| } | ||
| // Storage: System Account (r:1 w:1) | ||
| // Storage: ElectionProviderMultiPhase QueuedSolution (r:0 w:1) | ||
| fn finalize_signed_phase_accept_solution() -> Weight { | ||
| (48_439_000 as Weight) | ||
| (48_996_000 as Weight) | ||
| .saturating_add(RocksDbWeight::get().reads(1 as Weight)) | ||
| .saturating_add(RocksDbWeight::get().writes(2 as Weight)) | ||
| } | ||
| // Storage: System Account (r:1 w:1) | ||
| fn finalize_signed_phase_reject_solution() -> Weight { | ||
| (32_072_000 as Weight) | ||
| (32_508_000 as Weight) | ||
| .saturating_add(RocksDbWeight::get().reads(1 as Weight)) | ||
| .saturating_add(RocksDbWeight::get().writes(1 as Weight)) | ||
| } | ||
| // Storage: ElectionProviderMultiPhase SnapshotMetadata (r:0 w:1) | ||
| // Storage: ElectionProviderMultiPhase DesiredTargets (r:0 w:1) | ||
| // Storage: ElectionProviderMultiPhase Snapshot (r:0 w:1) | ||
| fn create_snapshot_internal(v: u32, t: u32, ) -> Weight { | ||
| (89_239_000 as Weight) | ||
| (96_001_000 as Weight) | ||
| // Standard Error: 1_000 | ||
| .saturating_add((320_000 as Weight).saturating_mul(v as Weight)) | ||
| .saturating_add((307_000 as Weight).saturating_mul(v as Weight)) | ||
| // Standard Error: 2_000 | ||
| .saturating_add((129_000 as Weight).saturating_mul(t as Weight)) | ||
| .saturating_add((133_000 as Weight).saturating_mul(t as Weight)) | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. these are more reasonable, the base weight is a bit less now, the component coefficients are a bit more, I presume in most cases the end up being the same thing, more or less. |
||
| .saturating_add(RocksDbWeight::get().writes(3 as Weight)) | ||
| } | ||
| // Storage: ElectionProviderMultiPhase SignedSubmissionIndices (r:1 w:1) | ||
|
|
@@ -241,11 +241,11 @@ impl WeightInfo for () { | |
| // Storage: ElectionProviderMultiPhase Snapshot (r:0 w:1) | ||
| // Storage: ElectionProviderMultiPhase CurrentPhase (r:0 w:1) | ||
| fn elect_queued(a: u32, d: u32, ) -> Weight { | ||
| (156_010_000 as Weight) | ||
| (100_505_000 as Weight) | ||
| // Standard Error: 6_000 | ||
| .saturating_add((1_579_000 as Weight).saturating_mul(a as Weight)) | ||
| // Standard Error: 9_000 | ||
| .saturating_add((299_000 as Weight).saturating_mul(d as Weight)) | ||
| .saturating_add((1_665_000 as Weight).saturating_mul(a as Weight)) | ||
| // Standard Error: 10_000 | ||
| .saturating_add((443_000 as Weight).saturating_mul(d as Weight)) | ||
| .saturating_add(RocksDbWeight::get().reads(6 as Weight)) | ||
| .saturating_add(RocksDbWeight::get().writes(8 as Weight)) | ||
| } | ||
|
|
@@ -256,9 +256,9 @@ impl WeightInfo for () { | |
| // Storage: ElectionProviderMultiPhase SignedSubmissionNextIndex (r:1 w:1) | ||
| // Storage: ElectionProviderMultiPhase SignedSubmissionsMap (r:0 w:1) | ||
| fn submit(c: u32, ) -> Weight { | ||
| (73_498_000 as Weight) | ||
| // Standard Error: 73_000 | ||
| .saturating_add((222_000 as Weight).saturating_mul(c as Weight)) | ||
| (74_088_000 as Weight) | ||
| // Standard Error: 59_000 | ||
| .saturating_add((187_000 as Weight).saturating_mul(c as Weight)) | ||
| .saturating_add(RocksDbWeight::get().reads(5 as Weight)) | ||
| .saturating_add(RocksDbWeight::get().writes(3 as Weight)) | ||
| } | ||
|
|
@@ -272,13 +272,13 @@ impl WeightInfo for () { | |
| fn submit_unsigned(v: u32, t: u32, a: u32, d: u32, ) -> Weight { | ||
| (0 as Weight) | ||
| // Standard Error: 5_000 | ||
| .saturating_add((2_028_000 as Weight).saturating_mul(v as Weight)) | ||
| // Standard Error: 11_000 | ||
| .saturating_add((124_000 as Weight).saturating_mul(t as Weight)) | ||
| .saturating_add((1_970_000 as Weight).saturating_mul(v as Weight)) | ||
| // Standard Error: 10_000 | ||
| .saturating_add((173_000 as Weight).saturating_mul(t as Weight)) | ||
| // Standard Error: 18_000 | ||
| .saturating_add((9_991_000 as Weight).saturating_mul(a as Weight)) | ||
| // Standard Error: 28_000 | ||
| .saturating_add((2_365_000 as Weight).saturating_mul(d as Weight)) | ||
| .saturating_add((9_783_000 as Weight).saturating_mul(a as Weight)) | ||
| // Standard Error: 27_000 | ||
| .saturating_add((2_224_000 as Weight).saturating_mul(d as Weight)) | ||
| .saturating_add(RocksDbWeight::get().reads(7 as Weight)) | ||
| .saturating_add(RocksDbWeight::get().writes(1 as Weight)) | ||
| } | ||
|
|
@@ -289,13 +289,13 @@ impl WeightInfo for () { | |
| fn feasibility_check(v: u32, t: u32, a: u32, d: u32, ) -> Weight { | ||
| (0 as Weight) | ||
| // Standard Error: 3_000 | ||
| .saturating_add((1_935_000 as Weight).saturating_mul(v as Weight)) | ||
| // Standard Error: 6_000 | ||
| .saturating_add((79_000 as Weight).saturating_mul(t as Weight)) | ||
| // Standard Error: 10_000 | ||
| .saturating_add((7_904_000 as Weight).saturating_mul(a as Weight)) | ||
| // Standard Error: 16_000 | ||
| .saturating_add((1_807_000 as Weight).saturating_mul(d as Weight)) | ||
| .saturating_add((1_910_000 as Weight).saturating_mul(v as Weight)) | ||
| // Standard Error: 7_000 | ||
| .saturating_add((111_000 as Weight).saturating_mul(t as Weight)) | ||
| // Standard Error: 13_000 | ||
| .saturating_add((7_741_000 as Weight).saturating_mul(a as Weight)) | ||
| // Standard Error: 19_000 | ||
| .saturating_add((1_844_000 as Weight).saturating_mul(d as Weight)) | ||
| .saturating_add(RocksDbWeight::get().reads(4 as Weight)) | ||
| } | ||
| } | ||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.