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

Improve paras runtime BenchBuilder api #4318

Merged
merged 19 commits into from
Dec 2, 2021

Conversation

emostov
Copy link
Contributor

@emostov emostov commented Nov 18, 2021

This PR primarily improves the BenchBuilder::build API by not requiring the user to left pad the dispute_sessions input to account for the cores used for backed_and_concluding.

@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Nov 18, 2021
@emostov emostov added B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit. labels Nov 18, 2021
@emostov emostov requested a review from drahnr November 24, 2021 19:46
Copy link
Contributor

@drahnr drahnr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few nits, mostly about docs and ergonomic API - other than that LGTM, it's going to be a big improvement!

@rphmeier
Copy link
Contributor

@emostov Will you address review comments? Nits & suggestions

@drahnr
Copy link
Contributor

drahnr commented Nov 30, 2021

Resolved the last nits.

@drahnr
Copy link
Contributor

drahnr commented Nov 30, 2021

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot
Copy link

Merge cancelled due to error. Error: Checks failed for e35c7bf

Copy link
Contributor

@drahnr drahnr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bot merge

@emostov
Copy link
Contributor Author

emostov commented Dec 2, 2021

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot paritytech-processbot bot merged commit de19db5 into master Dec 2, 2021
@paritytech-processbot paritytech-processbot bot deleted the zeke-adjust-paras-builder-api branch December 2, 2021 03:32
ordian added a commit that referenced this pull request Dec 3, 2021
* master: (23 commits)
  Fix path of the polkadot_injected docker image (#4463)
  update docs on `validation_upgrade_frequency` (#4460)
  Fix path of the polkadot sha256 (#4458)
  Fix cumulus companion CI job (#4451)
  Add doc about runtime version bump (#4418)
  Bump trybuild from 1.0.52 to 1.0.53 (#4455)
  companion for #10231 (#4306)
  trivial fix (#4441)
  Rococo: ability to programatically assign slots to teams (#3943)
  Add parent header hash to log (#4421)
  bump tx versions (#4447)
  Companion for substrate/10347 (#4413)
  Improve paras runtime `BenchBuilder` api (#4318)
  Add CI team to `CODEOWNERS` file (#4350)
  XCM Benchmarks for Generic Instructions (#3940)
  Announce only on releases (#4417)
  Companion for  10379 (EnsureOneOf) (#4405)
  add pallet-babe/std (#4438)
  update Cargo.lock
  Squashed 'bridges/' changes from 407bf44a8a..1602249f0a
  ...
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. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants