Skip to content

asset-hub-westend: use inmemorydb weights#8359

Merged
bkchr merged 6 commits intomasterfrom
alexggh/assethub_westend_read_write_weights
May 28, 2025
Merged

asset-hub-westend: use inmemorydb weights#8359
bkchr merged 6 commits intomasterfrom
alexggh/assethub_westend_read_write_weights

Conversation

@alexggh
Copy link
Copy Markdown
Contributor

@alexggh alexggh commented Apr 28, 2025

Modify asset-hub-westend to use the read/write costs in validate block on validator reference hardware, since these are
weights that guarantee the blocks will pass validation and make it on the relay chain.

This were obtained with the benchmark created here: #8069 and it contains all the optimisation brought by #8607.

Fixes: #7537

Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
alexggh added 2 commits May 26, 2025 10:48
Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
@alexggh alexggh changed the title asset-hub-westend: use validation memorydb weights asset-hub-westend: use inmemorydb weights May 26, 2025
@alexggh alexggh marked this pull request as ready for review May 26, 2025 08:05
@alexggh alexggh requested review from AndreiEres, athei and bkchr May 26, 2025 08:05
Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
@paritytech-workflow-stopper
Copy link
Copy Markdown

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

Copy link
Copy Markdown
Member

@athei athei left a comment

Choose a reason for hiding this comment

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

Nice

Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
@bkchr bkchr added the R0-no-crate-publish-required The change does not require any crates to be re-published. label May 27, 2025
@bkchr bkchr enabled auto-merge May 27, 2025 20:51
Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
@bkchr bkchr added this pull request to the merge queue May 28, 2025
Merged via the queue into master with commit 4b5a9ad May 28, 2025
181 checks passed
@bkchr bkchr deleted the alexggh/assethub_westend_read_write_weights branch May 28, 2025 08:33
pgherveou pushed a commit that referenced this pull request Jun 11, 2025
Modify asset-hub-westend to use the read/write costs in validate block
on validator reference hardware, since these are
weights that guarantee the blocks will pass validation and make it on
the relay chain.

This were obtained with the benchmark created here:
#8069 and it contains all
the optimisation brought by
#8607.

Fixes: #7537

---------

Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
alvicsam pushed a commit that referenced this pull request Oct 17, 2025
Modify asset-hub-westend to use the read/write costs in validate block
on validator reference hardware, since these are
weights that guarantee the blocks will pass validation and make it on
the relay chain.

This were obtained with the benchmark created here:
#8069 and it contains all
the optimisation brought by
#8607.

Fixes: #7537

---------

Signed-off-by: Alexandru Gheorghe <alexandru.gheorghe@parity.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

R0-no-crate-publish-required The change does not require any crates to be re-published.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Generate new weights as a new weights flavour InMemoryDBWeights

5 participants