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

benchmarking-cli: add --list-pallets and --all options #3395

Merged
merged 35 commits into from
Feb 21, 2024

Conversation

dastansam
Copy link
Contributor

@dastansam dastansam commented Feb 19, 2024

closes #2844

  • adds list-pallets option which prints all unique available pallets for benchmarking
./target/release/node benchmark pallet --list=pallets
  • adds all option which runs benchmarks for all available pallets and extrinsics (equivalent to --pallet * --extrinsic *)
./target/release/node benchmark pallet --all
  • use the list=pallets syntax in run_all_benchmarks.sh script

cc ggwpez

@dastansam dastansam marked this pull request as ready for review February 19, 2024 18:12
@dastansam dastansam changed the title benchmarking-cli: add --list-pallets option benchmarking-cli: add --list-pallets and --all options Feb 19, 2024
@ggwpez
Copy link
Member

ggwpez commented Feb 19, 2024

Usage looks fine, will give a closer look tomorrow 👍

@dastansam dastansam requested a review from ggwpez February 20, 2024 18:37
@dastansam
Copy link
Contributor Author

dastansam commented Feb 20, 2024

hey @ggwpez, comments are resolved. Not sure about the ListOutput name for enum, ListOption was another choice but it looked weird when wrapped with Option.

also, because of clap requiring requires_equal=true for default_missing_value, command to print pallets is:

./target/release/node benchmark pallet --list=pallets

@dastansam dastansam requested a review from ggwpez February 21, 2024 11:24
@ggwpez ggwpez added the T12-benchmarks This PR/Issue is related to benchmarking and weights. label Feb 21, 2024
@ggwpez
Copy link
Member

ggwpez commented Feb 21, 2024

It still needs a prdoc, then we can merge 😄

@dastansam
Copy link
Contributor Author

It still needs a prdoc, then we can merge 😄

added

prdoc/pr_3395.prdoc Outdated Show resolved Hide resolved
prdoc/pr_3395.prdoc Outdated Show resolved Hide resolved
@ggwpez ggwpez enabled auto-merge February 21, 2024 14:29
@ggwpez ggwpez added this pull request to the merge queue Feb 21, 2024
Merged via the queue into paritytech:master with commit 165d075 Feb 21, 2024
126 of 129 checks passed
bgallois pushed a commit to duniter/duniter-polkadot-sdk that referenced this pull request Mar 25, 2024
closes paritytech#2844 

- adds `list-pallets` option which prints all unique available pallets
for benchmarking
```bash
./target/release/node benchmark pallet --list=pallets
```
- adds `all` option which runs benchmarks for all available pallets and
extrinsics (equivalent to `--pallet * --extrinsic *`)
```bash
./target/release/node benchmark pallet --all
```

- use the `list=pallets` syntax in `run_all_benchmarks.sh` script

cc ggwpez

---------

Co-authored-by: Bastian Köcher <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T12-benchmarks This PR/Issue is related to benchmarking and weights.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

benchmarking-cli: add --list-pallets option
3 participants