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

Conversation

@NachoPal
Copy link
Contributor

@NachoPal NachoPal commented Oct 3, 2021

Related to: #627

Issue was introduced in: #498

Benchmarks stopped working because of this: https://github.com/paritytech/cumulus/pull/498/files#diff-a0359a0add814d66aac74493f7e17aefdcee21d3de1ae592e97059082b6feedfR336-R337 and the introduction of MinCandidates.

I had to fix register_as_candidate, leave_intent and new_session benchmarks.

In addition, I updated the benchmarks.sh script.

It fixes: #588

@NachoPal NachoPal requested a review from apopiak October 3, 2021 18:51
@NachoPal NachoPal merged commit 288a98d into release-statemine-v4 Oct 4, 2021
@NachoPal NachoPal deleted the nacho-statemine-v4-fix-benchmarks branch October 4, 2021 09:54
apopiak added a commit that referenced this pull request Oct 8, 2021
* register validators

* register_as_candidate & leave_intent fixed

* new_session benchmark fixed

* intent_leave_modified

* clean up

* clean up

* benchmark script updated

* update cargo.lock

* done

Co-authored-by: Alexander Popiak <[email protected]>
apopiak added a commit that referenced this pull request Oct 11, 2021
* bump runtime spec version

* remove applied runtime migrations

* bump transaction_version

necessary because of extrinsic API changes to pallet-xcm
paritytech/polkadot#3693

* Fix Benchmarks for Statemine-V4 release (#639)

* register validators

* register_as_candidate & leave_intent fixed

* new_session benchmark fixed

* intent_leave_modified

* clean up

* clean up

* benchmark script updated

* update cargo.lock

* done

Co-authored-by: Alexander Popiak <[email protected]>

* Version bump (#648)

* Version bump

fix #646

* Revert "Version bump"

This reverts commit 07517e0.

* Bump polkadot-collator version

* Update polkadot-parachains/Cargo.toml

Co-authored-by: Bastian Köcher <[email protected]>

* Update deps

* Bump version to 4.0.0

Co-authored-by: Bastian Köcher <[email protected]>

* cargo toml fix

* update deps and remove DisabledValidatorThreshold

* cargo +nightly fmt

* fix compile error

* fix client tests after Polkadot update

Co-authored-by: Ignacio Palacios <[email protected]>
Co-authored-by: Chevdor <[email protected]>
Co-authored-by: Bastian Köcher <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants