polkadot icon indicating copy to clipboard operation
polkadot copied to clipboard

[XCM] Replace Wild(All) for teleports and wherever else possible

Open ruseinov opened this issue 3 years ago • 2 comments

As the XCM v3 comes in we'd need to fix https://github.com/paritytech/polkadot/blob/53253de77f1649ecbb650df2ec3e29d0d1110a2d/xcm/pallet-xcm/src/lib.rs#L874 and similar occurrences with the new invariants of Wild.

Why? Because we are relying on coefficients to calculate weights, for example here and that turns out way more expensive, because we don't know an exact or approximate number of assets teleported.

cc @KiChjang

ruseinov avatar Sep 20 '22 09:09 ruseinov

It's not clear what coefficients mean from your description, I think what you meant was that we rely on the number of asset classes that is specified in the asset wildcard to calculate weights.

KiChjang avatar Sep 27 '22 07:09 KiChjang

@ruseinov Is this already done?

KiChjang avatar Feb 25 '23 05:02 KiChjang

I think this has already been done in cumulus.

KiChjang avatar Aug 15 '23 06:08 KiChjang