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

New weights #951

Merged
merged 2 commits into from
Feb 3, 2022
Merged

New weights #951

merged 2 commits into from
Feb 3, 2022

Conversation

NachoPal
Copy link
Contributor

@NachoPal NachoPal commented Feb 3, 2022

No description provided.

@NachoPal NachoPal added A0-pleasereview B0-silent Changes should not be mentioned in any release notes T7-system_parachains This PR/Issue is related to System Parachains. labels Feb 3, 2022
coderobe
coderobe reviewed Feb 3, 2022
View reviewed changes
Copy link
Contributor

@coderobe coderobe left a comment •
edited
Loading

Choose a reason for hiding this comment

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

I think that due to the recent DB cache size increase (128 -> 1024) many of these drift quite a bit, usually downwards

polkadot-parachains/statemine/src/weights/frame_system.rs Show resolved Hide resolved
polkadot-parachains/statemine/src/weights/frame_system.rs Show resolved Hide resolved
polkadot-parachains/statemine/src/weights/pallet_assets.rs Show resolved Hide resolved
polkadot-parachains/statemine/src/weights/pallet_assets.rs Show resolved Hide resolved
polkadot-parachains/statemine/src/weights/pallet_assets.rs Show resolved Hide resolved
polkadot-parachains/statemine/src/weights/pallet_assets.rs Show resolved Hide resolved
polkadot-parachains/statemine/src/weights/pallet_assets.rs Show resolved Hide resolved
@coderobe
Copy link
Contributor

coderobe commented Feb 3, 2022 •
edited
Loading

I've only highlighted some of the bigger changes - the recent DB cache size change has improved perf and thus lowered the weights across the board (i think?)

@shawntabrizi
Copy link
Member

Due to the recent DB cache size increase (128 -> 1024) many of these drift quite a bit, usually downwards

This change will not affect the weights you are pointing out at all.

@gilescope
Copy link
Contributor

gilescope commented Feb 3, 2022 •
edited
Loading

hmm, well we could rerun and check that the results are stable and it is not an aberration. Do we know of any PRs that should dramatically improved things?

gilescope
gilescope approved these changes Feb 3, 2022
View reviewed changes
Copy link
Contributor

@gilescope gilescope left a comment

Choose a reason for hiding this comment

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

Seems in line changes in general. As long as we are ok with 0 weight for remarks then looks reasonable. (remark_with_event is already 0 weighted)

polkadot-parachains/statemine/src/weights/pallet_assets.rs Show resolved Hide resolved
polkadot-parachains/statemine/src/weights/pallet_assets.rs Show resolved Hide resolved
polkadot-parachains/statemine/src/weights/pallet_assets.rs Show resolved Hide resolved
polkadot-parachains/statemine/src/weights/pallet_assets.rs Show resolved Hide resolved
polkadot-parachains/statemint/src/weights/frame_system.rs Show resolved Hide resolved
polkadot-parachains/westmint/src/weights/frame_system.rs Show resolved Hide resolved
polkadot-parachains/westmint/src/weights/pallet_multisig.rs Show resolved Hide resolved
@apopiak
Copy link
Contributor

apopiak commented Feb 3, 2022

just FYI: the Substrate weights went down when running the benchmarks with optimizations: paritytech/substrate#10692

NachoPal
NachoPal commented Feb 3, 2022
View reviewed changes
polkadot-parachains/statemine/src/weights/pallet_assets.rs Show resolved Hide resolved
polkadot-parachains/statemint/src/weights/frame_system.rs Show resolved Hide resolved
polkadot-parachains/statemint/src/weights/pallet_assets.rs Show resolved Hide resolved
polkadot-parachains/statemint/src/weights/pallet_proxy.rs Show resolved Hide resolved
polkadot-parachains/statemine/src/weights/pallet_uniques.rs Show resolved Hide resolved
polkadot-parachains/statemint/src/weights/pallet_uniques.rs Show resolved Hide resolved
polkadot-parachains/statemint/src/weights/pallet_uniques.rs Show resolved Hide resolved
polkadot-parachains/westmint/src/weights/pallet_uniques.rs Show resolved Hide resolved
polkadot-parachains/westmint/src/weights/pallet_uniques.rs Show resolved Hide resolved
polkadot-parachains/westmint/src/weights/frame_system.rs Show resolved Hide resolved
@NachoPal NachoPal merged commit be8b8e6 into release-parachains-v7.0.0 Feb 3, 2022
@NachoPal NachoPal deleted the nacho/new-weights-parachains-v7.0.0 branch February 3, 2022 19:55
gilescope pushed a commit that referenced this pull request Feb 9, 2022
@NachoPal @gilescope
new weights for release-parachains-v7.0.0 (#951)
d198a4c
paritytech-processbot bot pushed a commit that referenced this pull request Feb 10, 2022
@gilescope @NachoPal
new weights for release-parachains-v7.0.0 (#951) (#973)
cc7116d 
Co-authored-by: Ignacio Palacios <ignacio.palacios.santos@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@gilescope gilescope gilescope approved these changes

@coderobe coderobe coderobe left review comments

@ggwpez ggwpez ggwpez left review comments

@chevdor chevdor Awaiting requested review from chevdor

@EgorPopelyaev EgorPopelyaev Awaiting requested review from EgorPopelyaev

Assignees
No one assigned
Labels
B0-silent Changes should not be mentioned in any release notes T7-system_parachains This PR/Issue is related to System Parachains.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@NachoPal @coderobe @shawntabrizi @gilescope @apopiak @ggwpez