[AHM] Fresh weights for Polkadot#959
Merged
Merged
Conversation
Contributor
Author
|
/cmd bench --runtime bridge-hub-polkadot collectives-polkadot coretime-polkadot people-polkadot |
|
Command "bench --runtime bridge-hub-polkadot collectives-polkadot coretime-polkadot people-polkadot" has started 🚀 See logs here |
…dot collectives-polkadot coretime-polkadot people-polkadot'
|
Command "bench --runtime bridge-hub-polkadot collectives-polkadot coretime-polkadot people-polkadot" has finished ✅ See logs here DetailsSubweight results:
|
Contributor
Author
|
/cmd bench --runtime asset-hub-polkadot |
|
Command "bench --runtime asset-hub-polkadot" has started 🚀 See logs here |
Contributor
Author
|
/cmd bench --runtime asset-hub-polkadot |
|
Command "bench --runtime asset-hub-polkadot" has started 🚀 See logs here |
|
Command "bench --runtime asset-hub-polkadot" has finished ✅ See logs here DetailsSubweight results:
|
Contributor
Author
|
/cmd bench --runtime polkadot |
|
Command "bench --runtime polkadot" has started 🚀 See logs here |
|
Command "bench --runtime polkadot" has finished ✅ See logs here DetailsSubweight results:
|
Contributor
Author
|
/cmd bench --runtime asset-hub-polkadot --pallet pallet_election_provider_multi_block_verifier |
|
Command "bench --runtime asset-hub-polkadot --pallet pallet_election_provider_multi_block_verifier" has started 🚀 See logs here |
…ot --pallet pallet_election_provider_multi_block_verifier'
|
Command "bench --runtime asset-hub-polkadot --pallet pallet_election_provider_multi_block_verifier" has finished ✅ See logs here DetailsSubweight results:
|
Contributor
Author
|
/cmd bench --runtime asset-hub-polkadot --pallet pallet_election_provider_multi_block_verifier |
|
Command "bench --runtime asset-hub-polkadot --pallet pallet_election_provider_multi_block_verifier" has started 🚀 See logs here |
…ot --pallet pallet_election_provider_multi_block_verifier'
|
Command "bench --runtime asset-hub-polkadot --pallet pallet_election_provider_multi_block_verifier" has finished ✅ See logs here DetailsSubweight results:
|
ggwpez
reviewed
Oct 15, 2025
| // Minimum execution time: 28_130_262_000 picoseconds. | ||
| Weight::from_parts(28_781_353_000, 0) | ||
| .saturating_add(Weight::from_parts(0, 3215082)) | ||
| .saturating_add(T::DbWeight::get().reads(3042)) |
Contributor
There was a problem hiding this comment.
Yeah, we have the same weight for Kusama and it works fine. I also have tests in the runtime level to always sanity check the weights.
All weights are checked to be less than 70%:
runtimes/system-parachains/asset-hubs/asset-hub-polkadot/src/staking/mod.rs
Lines 606 to 706 in ba59f37
muharem
approved these changes
Oct 15, 2025
Contributor
Author
|
/cmd bench --runtime bridge-hub-polkadot --pallet pallet_bridge_messages |
|
Command "bench --runtime bridge-hub-polkadot --pallet pallet_bridge_messages" has started 🚀 See logs here |
…dot --pallet pallet_bridge_messages'
|
Command "bench --runtime bridge-hub-polkadot --pallet pallet_bridge_messages" has finished ✅ See logs here DetailsSubweight results:
|
kianenigma
approved these changes
Oct 15, 2025
bkontur
added a commit
that referenced
this pull request
Oct 19, 2025
Fixes BridgeHubPolkadot Snowbridge benchmarks leftover from: #959 catched by the test: https://github.com/polkadot-fellows/runtimes/actions/runs/18527148143/job/52800544994 <!-- Remember that you can run `/merge` to enable auto-merge in the PR --> <!-- Remember to modify the changelog. If you don't need to modify it, you can check the following box. Instead, if you have already modified it, simply delete the following line. --> - [X] Does not require a CHANGELOG entry
kianenigma
added a commit
to paritytech/polkadot-sdk
that referenced
this pull request
Nov 11, 2025
On top of just to see the weight diff with different bench config Solving issues in polkadot-fellows/runtimes#959 and polkadot-fellows/runtimes#963 --------- Co-authored-by: cmd[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.