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.

Sign up for GitHub

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

Jump to bottom

Integrate xcm benchmarked weights #552

Closed
ghzlatarev opened this issue May 16, 2022 · 1 comment · Fixed by #781, #822, #840 or #848
Closed

Integrate xcm benchmarked weights #552

ghzlatarev opened this issue May 16, 2022 · 1 comment · Fixed by #781, #822, #840 or #848
Assignees
@ghzlatarev
Labels
A-weights Area: Issues and PRs related to Substrate Weights A-xcm Area: Issues and PRs related to Cross-Consensus Messaging (XCM) P-medium Priority: Medium
Milestone
v3.4.0

Comments

@ghzlatarev
Copy link
Contributor

We can add benchmarked weights to xcm pallets now that paritytech/polkadot#4442 has landed
@ghzlatarev ghzlatarev added A-weights Area: Issues and PRs related to Substrate Weights A-xcm Area: Issues and PRs related to Cross-Consensus Messaging (XCM) labels May 16, 2022
@ghzlatarev ghzlatarev self-assigned this Jul 14, 2022
@ghzlatarev ghzlatarev added this to the v3.3.0 milestone Jul 14, 2022
@ghzlatarev ghzlatarev added the P-medium Priority: Medium label Jul 14, 2022
@ghzlatarev ghzlatarev modified the milestones: v3.3.0, v3.4.0 Aug 5, 2022
@ghzlatarev ghzlatarev mentioned this issue Aug 31, 2022
Merged
5 tasks
@ghzlatarev ghzlatarev reopened this Oct 8, 2022
@ghzlatarev ghzlatarev mentioned this issue Oct 8, 2022
Merged
6 tasks
@ghzlatarev
Copy link
Contributor Author

ghzlatarev commented Oct 8, 2022 •
edited
Loading

  • fix mock of xcm-tests to use new weights code
  • fix screenshot of dest_weight in QA docs, use moonbeam instead of dolphin
  • use max-assets parameter in benchmarking functions where possible
  • split hard-coded max-assets estimation for sender and receiver sides 2/100
  • ts script to stress-test holding register with 100 - 200 assets or more

This was referenced Oct 12, 2022
Closed
Closed
Merged
Merged
@ghzlatarev ghzlatarev modified the milestones: v3.4.0, v3.4.3 Oct 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ghzlatarev ghzlatarev

Labels
A-weights Area: Issues and PRs related to Substrate Weights A-xcm Area: Issues and PRs related to Cross-Consensus Messaging (XCM) P-medium Priority: Medium
Projects
None yet
Milestone
v3.4.0
1 participant
@ghzlatarev