This repository has been archived by the owner on Nov 15, 2023. It is now read-only.
update weights #5988
Merged
update weights #5988
Conversation
This file contains 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
coderobe
added
A0-please_review
This was referenced Sep 9, 2022
| Weight::from_ref_time(1_256_000 as u64) | ||
| // Standard Error: 42_000 | ||
| .saturating_add(Weight::from_ref_time(26_925_000 as u64).saturating_mul(s as u64)) | ||
| Weight::from_ref_time(11_217_000 as u64) |
There was a problem hiding this comment.
Looks like an asymptotic decrease actually.
| Weight::from_ref_time(921_000 as u64) | ||
| // Standard Error: 35_000 | ||
| .saturating_add(Weight::from_ref_time(21_922_000 as u64).saturating_mul(s as u64)) | ||
| Weight::from_ref_time(15_840_000 as u64) |
| Weight::from_ref_time(0 as u64) | ||
| // Standard Error: 62_000 | ||
| .saturating_add(Weight::from_ref_time(24_926_000 as u64).saturating_mul(s as u64)) | ||
| Weight::from_ref_time(10_761_000 as u64) |
There was a problem hiding this comment.
jump from 0 -> 10_761_000
| Weight::from_ref_time(0 as u64) | ||
| // Standard Error: 40_000 | ||
| .saturating_add(Weight::from_ref_time(7_752_000 as u64).saturating_mul(v as u64)) | ||
| Weight::from_ref_time(310_326_000 as u64) |
There was a problem hiding this comment.
jump from 0 -> 310_326_000
| Weight::from_ref_time(0 as u64) | ||
| // Standard Error: 43_000 | ||
| .saturating_add(Weight::from_ref_time(51_696_000 as u64).saturating_mul(n as u64)) | ||
| Weight::from_ref_time(15_720_000 as u64) |
There was a problem hiding this comment.
jump from 0 -> 15_720_000
ggwpez
reviewed
Sep 9, 2022
| Weight::from_ref_time(0 as u64) | ||
| // Standard Error: 28_000 | ||
| .saturating_add(Weight::from_ref_time(29_706_000 as u64).saturating_mul(b as u64)) | ||
| Weight::from_ref_time(13_392_000 as u64) |
There was a problem hiding this comment.
jump from 0 -> 13_392_000
| // Standard Error: 0 | ||
| .saturating_add(Weight::from_ref_time(165_000 as u64).saturating_mul(m as u64)) | ||
| .saturating_add(Weight::from_ref_time(25_000 as u64).saturating_mul(m as u64)) |
| // Standard Error: 0 | ||
| .saturating_add(Weight::from_ref_time(163_000 as u64).saturating_mul(m as u64)) | ||
| .saturating_add(Weight::from_ref_time(27_000 as u64).saturating_mul(m as u64)) |
| Weight::from_ref_time(125_000 as u64) | ||
| // Standard Error: 37_000 | ||
| .saturating_add(Weight::from_ref_time(23_169_000 as u64).saturating_mul(s as u64)) | ||
| Weight::from_ref_time(13_757_000 as u64) |
There was a problem hiding this comment.
significant jump 125_000 -> 13_757_000
| Weight::from_ref_time(0 as u64) | ||
| // Standard Error: 37_000 | ||
| .saturating_add(Weight::from_ref_time(7_545_000 as u64).saturating_mul(v as u64)) | ||
| Weight::from_ref_time(105_025_000 as u64) |
There was a problem hiding this comment.
significant jump 0 -> 105_025_000
| Weight::from_ref_time(0 as u64) | ||
| // Standard Error: 41_000 | ||
| .saturating_add(Weight::from_ref_time(49_451_000 as u64).saturating_mul(n as u64)) | ||
| Weight::from_ref_time(21_631_000 as u64) |
There was a problem hiding this comment.
jump from 0 -> 21_631_000
| Weight::from_ref_time(0 as u64) | ||
| // Standard Error: 25_000 | ||
| .saturating_add(Weight::from_ref_time(51_756_000 as u64).saturating_mul(n as u64)) | ||
| Weight::from_ref_time(18_891_000 as u64) |
| @@ -109,11 +109,11 @@ impl<T: frame_system::Config> pallet_election_provider_multi_phase::WeightInfo f | |||
| /// The range of component `a` is `[500, 800]`. | |||
| /// The range of component `d` is `[200, 400]`. | |||
| fn elect_queued(a: u32, d: u32, ) -> Weight { | |||
| Weight::from_ref_time(9_172_000 as u64) | |||
| Weight::from_ref_time(0 as u64) | |||
| Weight::from_ref_time(0 as u64) | ||
| // Standard Error: 14_000 | ||
| .saturating_add(Weight::from_ref_time(18_621_000 as u64).saturating_mul(k as u64)) | ||
| Weight::from_ref_time(19_773_000 as u64) |
There was a problem hiding this comment.
jump from 0 -> 19_773_000
| Weight::from_ref_time(0 as u64) | ||
| // Standard Error: 28_000 | ||
| .saturating_add(Weight::from_ref_time(48_794_000 as u64).saturating_mul(n as u64)) | ||
| Weight::from_ref_time(18_538_000 as u64) |
There was a problem hiding this comment.
jump from 0 -> 18_538_000
|
I am surprised not to see Kusama in here |
The benchmarking has some shortcomings for very short functions, like |
ggwpez
approved these changes
Sep 12, 2022
2 tasks
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
🏋️♀️