Skip to content

Benchmark All Pallets#1585

Merged
sam0x17 merged 12 commits intodevnet-readyfrom
benchmark-all-pallets
May 5, 2025
Merged

Benchmark All Pallets#1585
sam0x17 merged 12 commits intodevnet-readyfrom
benchmark-all-pallets

Conversation

@JohnReedV
Copy link
Contributor

@JohnReedV JohnReedV commented May 1, 2025

Description

This PR expands the validate-benchmarks action to

  • Validate benchmarks on multiple pallets
  • Use the dedicated benchmark runner instead of SubtensorCI

Also updates extrinsic weights for newly benchmarked pallets

@JohnReedV JohnReedV added the skip-cargo-audit This PR fails cargo audit but needs to be merged anyway label May 1, 2025
@JohnReedV JohnReedV marked this pull request as ready for review May 1, 2025 20:41
@JohnReedV JohnReedV added skip-validate-benchmarks This PR fails validate benchmarks but needs to be merged anyway run-bittensor-e2e-tests and removed skip-validate-benchmarks This PR fails validate benchmarks but needs to be merged anyway labels May 1, 2025
sam0x17
sam0x17 previously approved these changes May 2, 2025
@JohnReedV JohnReedV dismissed sam0x17’s stale review May 2, 2025 00:34

The merge-base changed after approval.

sam0x17
sam0x17 previously approved these changes May 2, 2025
@JohnReedV JohnReedV dismissed sam0x17’s stale review May 2, 2025 00:41

The merge-base changed after approval.

@JohnReedV JohnReedV force-pushed the benchmark-all-pallets branch from ddb1fbc to f9fa3cf Compare May 2, 2025 16:28
@sam0x17 sam0x17 merged commit 9d9f384 into devnet-ready May 5, 2025
53 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-cargo-audit This PR fails cargo audit but needs to be merged anyway

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants