-
Notifications
You must be signed in to change notification settings - Fork 589
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
Update Dex Benchmaking #1588
Update Dex Benchmaking #1588
Conversation
/bench runtime module module_dex |
Finished benchmark for branch: fer-bench-audits Benchmark: Benchmark Runtime Module cargo run --release --color=never --bin=acala --features=runtime-benchmarks --features=with-mandala-runtime -- benchmark --chain=dev --steps=50 --repeat=20 --pallet=module_dex --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --output=./modules/dex/src/weights.rs --template=./templates/module-weight-template.hbs ResultsPallet: "module_dex", Extrinsic: "enable_trading_pair", Lowest values: [], Highest values: [], Steps: 50, Repeat: 20
|
/bench runtime karura module_dex |
Finished benchmark for branch: fer-bench-audits Benchmark: Benchmark Runtime Karura Module cargo run --release --color=never --bin=acala --features=runtime-benchmarks --features=with-karura-runtime -- benchmark --chain=karura-dev --steps=50 --repeat=20 --pallet=module module_dex --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --template=./templates/runtime-weight-template.hbs --output=./runtime/karura/src/weights/ Results
Compiling acala-cli v2.0.0 (/app/git/Acala/node/cli) warning: unused import: warning: unused imports: warning: unused import: Compiling acala-rpc v2.0.0 (/app/git/Acala/rpc) warning: unused import: warning: 2 warnings emitted Compiling acala-service v2.0.0 (/app/git/Acala/node/service) Compiling polkadot-cli v0.9.12 (https://github.com/paritytech//polkadot?rev=5feed981cf0fe671447eabaa9c0d235a8dd0003e#5feed981) Compiling acala v2.0.0 (/app/git/Acala/node) USAGE: For more information try --help |
Finished benchmark for branch: fer-bench-audits Benchmark: Benchmark Runtime Karura Module cargo run --release --color=never --bin=acala --features=runtime-benchmarks --features=with-karura-runtime -- benchmark --chain=karura-dev --steps=50 --repeat=20 --pallet=module_dex --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --template=./templates/runtime-weight-template.hbs --output=./runtime/karura/src/weights/ ResultsPallet: "module_dex", Extrinsic: "enable_trading_pair", Lowest values: [], Highest values: [], Steps: 50, Repeat: 20
|
/bench runtime karura module_dex |
Finished benchmark for branch: fer-bench-audits Benchmark: Benchmark Runtime Karura Module cargo run --release --color=never --bin=acala --features=runtime-benchmarks --features=with-karura-runtime -- benchmark --chain=karura-dev --steps=50 --repeat=20 --pallet=module_dex --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --template=./templates/runtime-weight-template.hbs --output=./runtime/karura/src/weights/ ResultsPallet: "module_dex", Extrinsic: "enable_trading_pair", Lowest values: [], Highest values: [], Steps: 50, Repeat: 20
|
|
Locally I get that output (which I believe to be correct), but the benchbot doesn't generate the same reads and write scheme, obviously the numbers are different but shouldn't the reads and writes be identical across machines?
bench-bot also gets the same results it appears but isn't updating weights file? |
/bench runtime karura * |
Finished benchmark for branch: fer-bench-audits Benchmark: Benchmark Runtime Karura Module cargo run --release --color=never --bin=acala --features=runtime-benchmarks --features=with-karura-runtime -- benchmark --chain=karura-dev --steps=50 --repeat=20 --pallet=* --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --template=./templates/runtime-weight-template.hbs --output=./runtime/karura/src/weights/ ResultsPallet: "module_nft", Extrinsic: "create_class", Lowest values: [], Highest values: [], Steps: 50, Repeat: 20
|
/bench runtime karura module_dex |
Finished benchmark for branch: fer-bench-audits Benchmark: Benchmark Runtime Karura Module cargo run --release --color=never --bin=acala --features=runtime-benchmarks --features=with-karura-runtime -- benchmark --chain=karura-dev --steps=50 --repeat=20 --pallet=module_dex --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --template=./templates/runtime-weight-template.hbs --output=./runtime/karura/src/weights/ ResultsPallet: "module_dex", Extrinsic: "enable_trading_pair", Lowest values: [], Highest values: [], Steps: 50, Repeat: 20
|
…hmarks --features=with-karura-runtime -- benchmark --chain=karura-dev --steps=50 --repeat=20 --pallet=module_dex --extrinsic=* --execution=wasm --wasm-execution=compiled --heap-pages=4096 --template=./templates/runtime-weight-template.hbs --output=./runtime/karura/src/weights/
/bench runtime karura module_dex |
Finished benchmark for branch: fer-bench-audits Benchmark: Benchmark Runtime Karura Module cargo run --release --color=never --bin=acala --features=runtime-benchmarks --features=with-karura-runtime -- benchmark --chain=karura-dev --steps=50 --repeat=20 --pallet=module_dex --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --template=./templates/runtime-weight-template.hbs --output=./runtime/karura/src/weights/ ResultsPallet: "module_dex", Extrinsic: "enable_trading_pair", Lowest values: [], Highest values: [], Steps: 50, Repeat: 20
|
…hmarks --features=with-karura-runtime -- benchmark --chain=karura-dev --steps=50 --repeat=20 --pallet=module_dex --extrinsic=* --execution=wasm --wasm-execution=compiled --heap-pages=4096 --template=./templates/runtime-weight-template.hbs --output=./runtime/karura/src/weights/
/bench runtime mandala module_dex |
Finished benchmark for branch: fer-bench-audits Benchmark: Benchmark Runtime Mandala Module cargo run --release --color=never --bin=acala --features=runtime-benchmarks --features=with-mandala-runtime -- benchmark --chain=dev --steps=50 --repeat=20 --pallet=module_dex --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --template=./templates/runtime-weight-template.hbs --output=./runtime/mandala/src/weights/ ResultsPallet: "module_dex", Extrinsic: "enable_trading_pair", Lowest values: [], Highest values: [], Steps: 50, Repeat: 20
|
…hmarks --features=with-mandala-runtime -- benchmark --chain=dev --steps=50 --repeat=20 --pallet=module_dex --extrinsic=* --execution=wasm --wasm-execution=compiled --heap-pages=4096 --template=./templates/runtime-weight-template.hbs --output=./runtime/mandala/src/weights/
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the weight changes looks reasonable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fixes swap benchmarks, I can restore the weights files if we don't want to merge that right now