Merged
Conversation
girazoki
reviewed
Feb 20, 2025
girazoki
reviewed
Feb 20, 2025
girazoki
reviewed
Feb 20, 2025
girazoki
reviewed
Feb 20, 2025
girazoki
reviewed
Feb 20, 2025
...container-chains/runtime-templates/frontier/src/weights/xcm/pallet_xcm_benchmarks_generic.rs
Outdated
Show resolved
Hide resolved
girazoki
reviewed
Feb 20, 2025
girazoki
reviewed
Feb 20, 2025
girazoki
reviewed
Feb 20, 2025
Contributor
WASM runtime size check:Compared to target branchdancebox runtime: 1500 KB (+72 KB) flashbox runtime: 888 KB (+40 KB) dancelight runtime: 2260 KB (+80 KB) container chain template simple runtime: 1176 KB (+44 KB) container chain template frontier runtime: 1452 KB (+36 KB) |
Contributor
ParthDesai
reviewed
Mar 5, 2025
girazoki
approved these changes
Mar 5, 2025
Contributor
girazoki
left a comment
There was a problem hiding this comment.
Changes look good to me, my only comment is that we need to benchmark pallet_transaction_payment and also the new extensions. IF not in this PR, in another, but we should create an issue at least for this
ParthDesai
reviewed
Mar 5, 2025
ParthDesai
reviewed
Mar 5, 2025
ParthDesai
reviewed
Mar 5, 2025
ParthDesai
reviewed
Mar 5, 2025
chains/orchestrator-paras/runtime/dancebox/src/tests/common/xcm/core_buyer.rs
Show resolved
Hide resolved
nanocryk
approved these changes
Mar 5, 2025
Contributor
nanocryk
left a comment
There was a problem hiding this comment.
LGTM outside of others comments
Agusrodri
reviewed
Mar 5, 2025
Agusrodri
approved these changes
Mar 5, 2025
Contributor
Agusrodri
left a comment
There was a problem hiding this comment.
LGTM. Great job @tmpolaczyk!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Upgrade polkadot-sdk to stable2412
main PRs
XCM v5
pallet identity migration - our first multiblock migration
extension weights - increases weight and proof size of all extrinsics by taking into account validations like signature, nonce, age, etc
CandidateReceiptV2
not important but good to know
extension weights - terminology change: unsigned => bare. Also refactors traits such as CreateSignedTransaction
deprecate RuntimeString - Remove sp_runtime::RuntimeString and replace with Cow<'static, str> or String depending on use case
system_version - rename RuntimeVersion::state_version to system_version. Setting system_version = 2 will make extrinsics use StateVersion::V1 instead of V0, but we don't do that, and neither does parity
remove ttl field - not removed, instead deprecated and ignored
new types in pallet config
cumulus core selector - type SelectCore
generic slashing side-effects - type DoneSlashHandler
pallet identity migration - type UsernameDeposit + type UsernameGracePeriod
pallet treasury generic BlockNumberProvider - type BlockNumberProvider
extension weights - type ExtensionsWeightInfo
Full PR list
#889