Refactor property writing + dev mode enhancements #1000
Conversation
mrshiposha
added
CI-collator-selection
uandysmith
force-pushed
the
fix/minting-prop-weight
branch
from
5c37987 to
d2c9363
Compare
mrshiposha
changed the title
| // Measured: `667` | ||
| // Estimated: `4325` | ||
| // Minimum execution time: 7_344_000 picoseconds. | ||
| Weight::from_parts(7_578_000, 4325) |
There was a problem hiding this comment.
Хм, интересно почему. У нас для поиска читалась какая-то большая информация для токенов, которая однако погружалась в коде инициализации бенчмарка? Да вроде не похоже...
| @@ -52,10 +52,10 @@ pub const MAX_COLLATORS: u32 = 10; | |||
| pub const SESSION_LENGTH: BlockNumber = HOURS; | |||
|
|
|||
| // Targeting 0.1 UNQ per transfer | |||
| pub const WEIGHT_TO_FEE_COEFF: u64 = /*<weight2fee>*/76_840_511_488_584_762/*</weight2fee>*/; | |||
| pub const WEIGHT_TO_FEE_COEFF: u64 = /*<weight2fee>*/77_334_604_063_436_322/*</weight2fee>*/; | |||
There was a problem hiding this comment.
Изменение во втором и третьем знаке! 🎉
| @@ -90,7 +90,7 @@ evm_stubs: UniqueFungible UniqueNFT UniqueRefungible UniqueRefungibleToken Contr | |||
|
|
|||
| .PHONY: _bench | |||
| _bench: | |||
| cargo run --release --features runtime-benchmarks,$(RUNTIME) -- \ | |||
| cargo run --profile production --features runtime-benchmarks,$(RUNTIME) -- \ | |||
There was a problem hiding this comment.
Я подумал, а зачем мы вообще это кучу раз компилируем...
У нас там перед каждым бенчмарком палеты ноду снова пересобирает, и я только что понял что она пересобирает - у нас изменяется weights.rs, и оно собирает ноду с новыми весами.
По идее тут нужно один раз ноду собрать, и потом вызывать ./unique-collator benchmark ...
Что тут однако не выйдет сделать, поскольку у нас рекурсивный make.
Сейчас unactionable, но способ оптимизации ясен.
| pub fn has_value(&self) -> bool { | ||
| self.value.is_some() | ||
| } | ||
|
|
||
| fn compute_value_if_not_already(&mut self) { |
There was a problem hiding this comment.
Длинно и не так понятно)
Может force() назвать, как в FP принято?
There was a problem hiding this comment.
Мб force_value() тогда?
| nesting_budget, | ||
| )?; | ||
|
|
||
| Ok(is_bundle_owner) |
There was a problem hiding this comment.
| Ok(is_bundle_owner) | |
| <PalletStructure<T>>::check_indirectly_owned( | |
| maybe_owner.clone(), | |
| self.id, | |
| token, | |
| None, | |
| nesting_budget, | |
| ) |
No description provided.