Update polkadot to v0.9.43 #968
Conversation
Signed-off-by: Yaroslav Bolyukin <[email protected]>
Signed-off-by: Yaroslav Bolyukin <[email protected]>
Signed-off-by: Yaroslav Bolyukin <[email protected]>
Signed-off-by: Yaroslav Bolyukin <[email protected]>
Signed-off-by: Yaroslav Bolyukin <[email protected]>
Signed-off-by: Yaroslav Bolyukin <[email protected]>
| @@ -31,7 +34,10 @@ parameter_types! { | |||
| pub const GasPerSecond: u64 = ReadsPerSecond::get() * 2100; | |||
| pub const WeightTimePerGas: u64 = WEIGHT_REF_TIME_PER_SECOND / GasPerSecond::get(); | |||
|
|
|||
| pub const WeightPerGas: Weight = Weight::from_parts(WeightTimePerGas::get(), 0); | |||
| pub const BytesReadPerSecond: u64 = ReadsPerSecond::get() * EVM_SLOAD_PROOF_SIZE; | |||
| pub const ProofSizePerGas: u64 = 0; //WEIGHT_REF_TIME_PER_SECOND / GasPerSecond::get(); | |||
There was a problem hiding this comment.
Такая простая формула тут не работает по разным причинам, нормальная токеномика будет проставлена после разбирательств с frontier
There was a problem hiding this comment.
=> Будет исправлено после задачи по бенчмаркингу открытия EVM контекста
CertainLach
force-pushed
the
feature/update-polkadot-v0.9.43
branch
from
2b81a36
to
1b4b057
Compare
| @@ -1111,32 +1133,42 @@ where | |||
|
|
|||
| #[cfg(feature = "pov-estimate")] | |||
| backend, | |||
| eth_backend: frontier_backend.clone(), | |||
| // eth_backend, | |||
| overrides.clone(), | ||
| eth_backend, | ||
| 3, | ||
| 0, |
| @@ -178,6 +217,9 @@ mod pallet { | |||
| } else { | |||
| <MinGasPriceOverride<T>>::kill(); | |||
| } | |||
| // This code should not be called in production, but why keep development in the | |||
There was a problem hiding this comment.
but why keep development
Мб имелось в виду "we keep"? Ибо это не звучит как вопрос.
There was a problem hiding this comment.
Это вопрос, просто риторический)
There was a problem hiding this comment.
А кто знает ответ?)
Чет не пойму что тут к чему :D
| @@ -20,6 +20,9 @@ use up_common::constants::*; | |||
|
|
|||
| pub type CrossAccountId = pallet_evm::account::BasicCrossAccountId<Runtime>; | |||
|
|
|||
| // Assuming PoV size per read is 96 bytes: 16 for twox128(Evm), 16 for twox128(Storage), 32 for storage key, and 32 for storage value | |||
| const EVM_SLOAD_PROOF_SIZE: u64 = 96; | |||
|
|
|||
| // ~~Assuming slowest ethereum opcode is SSTORE, with gas price of 20000 as our worst case~~ | |||
There was a problem hiding this comment.
Коль это "very wrong", то мб еще отдельно скажем, что это TODO и будет сделано "тогда-то"
There was a problem hiding this comment.
Это уже сделано, тут объяснена история принятия такого решения, потому что оно не интуитивное.
В мунбиме и прочих тут константа из головы, у нас стоимость описана формулой
There was a problem hiding this comment.
Просто из коммента не ясно что оно уже исправлено, а ощущение, что это надо исправить
No description provided.