feat: time-based UD
Merge request reports
Activity
changed milestone to %runtime-600
added RN-runtime label
added 19 commits
-
3c300c73...81f4a877 - 18 commits from branch
master
- 49bb7063 - feat: time-based UD
-
3c300c73...81f4a877 - 18 commits from branch
requested review from @HugoTrentesaux
- Resolved by Pascal Engélibert
- Resolved by Cédric Moreau
- Resolved by Pascal Engélibert
- Resolved by Pascal Engélibert
- Resolved by Pascal Engélibert
- Resolved by Cédric Moreau
- Resolved by Cédric Moreau
requested review from @c-geek and removed review request for @HugoTrentesaux
- Resolved by Cédric Moreau
Il reste un soucis, je ne sais pas encore si cela vient de tes modifications où si le bug était présent avant, j'essaye de révéler l'anomlie via un TU.
- Resolved by Pascal Engélibert
- Resolved by Cédric Moreau
Je vais aussi supprimer
MembersStorageIter
qui n'est jamais utilisé.Edit: en fait il est utilisé pour trouver le compte de test pour un benchmark. C'est un peu overkill mais ça va être long de trouver un autre moyen. Ce sera pour plus tard.
Edited by Pascal Engélibertenabled an automatic merge when the pipeline for 0728b9df succeeds
142 147 // GENESIS 143 148 144 149 #[pallet::genesis_config] 145 pub struct GenesisConfig<T: Config> { 146 pub first_reeval: T::BlockNumber, 147 pub first_ud: BalanceOf<T>, 150 pub struct GenesisConfig<T: Config> 151 where 152 <T as pallet_timestamp::Config>::Moment: MaybeSerializeDeserialize, 153 { 154 /// If None, it will be set to one period after the first block with a timestamp 155 pub first_reeval: Option<T::Moment>, 156 /// If None, it will be set to one period after the first block with a timestamp 157 pub first_ud: Option<T::Moment>, Il me fallait un moyen de définir la date du premier DU, et cette date ne changera jamais dans l'histoire de la blockchain (il existe un unique premier DU). Le montant du premier DU peut juste être vu comme le montant du DU au moment du genesis. Si on l'appelle
first_ud
, pourquoi ne pas appeler les certifications genesisfirst_certs
? C'est juste un instantané d'une variable qui change dans le temps. Donc j'ai trouvé plus clair d'échanger le nom plutôt que de trouver un nom plus long où on se demande si first_ud est la date ou le montant, et où il faut trouver first_ud_moment pour le deviner par élimination. Alors que "first" est plus explicitement temporel.
mentioned in merge request !172 (merged)
mentioned in issue #223