Skip to content
Snippets Groups Projects

feat: time-based UD

Merged Pascal Engélibert requested to merge tuxmain/time-ud into master
1 unresolved thread

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Cédric Moreau
  • Cédric Moreau
  • Voilà une 1ère passe de review, il faut encore que je creuse pour vérifier le reste.

  • Cédric Moreau
  • Pascal Engélibert requested review from @c-geek and removed review request for @HugoTrentesaux

    requested review from @c-geek and removed review request for @HugoTrentesaux

  • added 1 commit

    Compare with previous version

  • Cédric Moreau
  • C'est bon, je pense avoir terminé.

  • 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élibert
  • Cédric Moreau resolved all threads

    resolved all threads

  • Cédric Moreau approved this merge request

    approved this merge request

  • Cédric Moreau resolved all threads

    resolved all threads

  • Cédric Moreau unapproved this merge request

    unapproved this merge request

  • Cédric Moreau resolved all threads

    resolved all threads

  • Cédric Moreau approved this merge request

    approved this merge request

  • added 1 commit

    Compare with previous version

  • Pascal Engélibert enabled an automatic merge when the pipeline for 0728b9df succeeds

    enabled an automatic merge when the pipeline for 0728b9df succeeds

  • Pascal Engélibert canceled the automatic merge

    canceled the automatic merge

  • 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>,
    • Pourquoi first_ud qui était un BalanceOf<T> (donc un montant, je l'ai renommé first_ud_value dans ma branche hugo-dev) est devenu un Option<T::Moment>, c'est-à-dire un instant ?

    • 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 genesis first_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.

    • Please register or sign in to reply
  • Est-ce que tu auras le temps de mettre à jour les benchmarks correspondants ?

  • Hugo Trentesaux mentioned in merge request !172 (merged)

    mentioned in merge request !172 (merged)

  • Hugo Trentesaux mentioned in issue #223

    mentioned in issue #223

  • Please register or sign in to reply
    Loading