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

Update Benchmarks and Weights for Delayed Proxy#6811

Merged
shawntabrizi merged 5 commits intogav-delay-proxyfrom
shawntabrizi-delay-proxy-weights
Aug 4, 2020
Merged

Update Benchmarks and Weights for Delayed Proxy#6811
shawntabrizi merged 5 commits intogav-delay-proxyfrom
shawntabrizi-delay-proxy-weights

Conversation

@shawntabrizi
Copy link
Member

This PR updates the benchmarks and weights for delayed proxy in order to include announcements into the weight formula as a parameter

@shawntabrizi shawntabrizi merged commit 4094a17 into gav-delay-proxy Aug 4, 2020
@shawntabrizi shawntabrizi deleted the shawntabrizi-delay-proxy-weights branch August 4, 2020 10:35
@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Aug 4, 2020
ghost pushed a commit that referenced this pull request Aug 23, 2020
* Time-delay proxies.

* Tests

* Initial couple of benchmarks

* Fix up runtime

* Last couple of benchmarks

* Tests

* Docs

* Migration

* add tests to proxy benchmarks

* generated benchmarks, not integrated

* Fix weight trait

* integrate weightinfo

* default weight

* Grumble

* Deduplication, split proxy from announced_proxy and don't require reauthentication

* Fix

* Remoe superfluous

* Typos

* Indent

* Fix

* Fixes

* rename 'proxy_announced' -> 'announced_proxy'

* flip rename

* comments and spacing

* fix proxy_announced

* remove unneeded `execute` marker

* Avoid unneeded changes to extrinsic indices

* Cleanup

* Fixes

* Update Benchmarks and Weights for Delayed Proxy (#6811)

* update bechmarks to parameterize announcements

* remove announcement param from proxy

* Update pallet_proxy.rs

* Update weights

* Bump runtime

* Fix benchmark

Co-authored-by: Shawn Tabrizi <shawntabrizi@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A0-please_review Pull request needs code review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants