-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Conversation
…tech/substrate into kiz-call-estimator
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - I think we need to update weights
/benchmark runtime pallet pallet-staking |
Benchmark Runtime Pallet for branch "kiz-call-estimator" with command cargo run --quiet --release --features=runtime-benchmarks --manifest-path=bin/node/cli/Cargo.toml -- benchmark --chain=dev --steps=50 --repeat=20 --pallet=pallet-staking --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --output=./frame//src/weights.rs --template=./.maintain/frame-weight-template.hbs Results
|
… kiz-call-estimator
@@ -656,6 +656,19 @@ where | |||
} | |||
} | |||
|
|||
impl<T: Config, AnyCall: GetDispatchInfo + Encode> EstimateCallFee<AnyCall, BalanceOf<T>> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be called MaxCallFee
?
Estimate implies to me it could be more or less, and that it is actually estimating something, but really it can only be less than this fee. This is just the max.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm trying to take into account that other implementations might differ, and might be more or less. The implementation by transaction-payment
is actually accurate to the point (excluding tip).
But the trait name should not be moulded by the implementation detail, IMO in any case.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like a great approach
Co-authored-by: Zeke Mostov <32168567+emostov@users.noreply.github.com>
/benchmark runtime pallet pallet-staking |
Benchmark Runtime Pallet for branch "kiz-call-estimator" with command cargo run --quiet --release --features=runtime-benchmarks --manifest-path=bin/node/cli/Cargo.toml -- benchmark --chain=dev --steps=50 --repeat=20 --pallet=pallet-staking --extrinsic="*" --execution=wasm --wasm-execution=compiled --heap-pages=4096 --output=./frame//src/weights.rs --template=./.maintain/frame-weight-template.hbs Results
|
bot merge |
Waiting for commit status. |
fixes #9247
This will allow us to properly know roughly how much fee will be consumed for a given call within a dispatch, and store it. We then use this later to give an appropriate reward to the signed submissions.
TODO:
CallFeeEstimator
ineleciton-provider
that just assumesscore[0]
(or some fixed amount) is the correct amountsubstrate/pull/9395
) polkadot#3465 the companion of this PR, update rewards to a small value, like1 * DOLLAR
polkadot companion: paritytech/polkadot#3465