Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Auto-update of all weights for 2025-02-11-1739295016 #7547

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

paritytech-cmd-bot-polkadot-sdk[bot]

Auto-update of all weights for 2025-02-11-1739295016.

Subweight results:

@paritytech-cmd-bot-polkadot-sdk paritytech-cmd-bot-polkadot-sdk bot added the R0-silent Changes should not be mentioned in any release notes label Feb 12, 2025
@paritytech-cmd-bot-polkadot-sdk paritytech-cmd-bot-polkadot-sdk bot requested review from a team February 12, 2025 08:01
@mordamax mordamax marked this pull request as ready for review February 13, 2025 23:50
@mordamax mordamax requested review from acatangiu, cheme and a team as code owners February 13, 2025 23:50
@paritytech-review-bot paritytech-review-bot bot requested a review from a team February 13, 2025 23:52
Comment on lines +117 to +123
// Minimum execution time: 1_026_794_000 picoseconds.
Weight::from_parts(1_062_697_000, 0)
.saturating_add(Weight::from_parts(0, 134587))
// Standard Error: 1_263_527
.saturating_add(Weight::from_parts(454_948_671, 0).saturating_mul(u.into()))
// Standard Error: 1_263_527
.saturating_add(Weight::from_parts(527_131_211, 0).saturating_mul(h.into()))
// Standard Error: 2_093_815
.saturating_add(Weight::from_parts(11_112_489, 0).saturating_mul(c.into()))
// Standard Error: 5_157_176
.saturating_add(Weight::from_parts(211_006_910, 0).saturating_mul(u.into()))
// Standard Error: 5_157_176
.saturating_add(Weight::from_parts(344_658_237, 0).saturating_mul(h.into()))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see that the weight now is independent of the c (whether there was a code upgrade) param. And the base ref time went up substantially (from 200us to 1ms - 5x). I wonder what is causing this?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was last updated 6 months ago... this is the kind of issue we try to solve here. Have frequent weight updates to see early when things change.
For the change itself: I dont know

@paritytech-workflow-stopper
Copy link

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/13331161319
Failed job name: test-linux-stable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants