Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
Duniter v2S
Manage
Activity
Members
Labels
Plan
Issues
Issue boards
Milestones
Code
Merge requests
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Build
Pipelines
Jobs
Pipeline schedules
Artifacts
Deploy
Releases
Container Registry
Model registry
Monitor
Service Desk
Analyze
Value stream analytics
Contributor analytics
CI/CD analytics
Repository analytics
Model experiments
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
nodes
rust
Duniter v2S
Merge requests
!44
ref(pallet-ud): remove params UdFirstReeval & UdReevalPeriodInBlocks
Code
Review changes
Check out branch
Download
Patches
Plain diff
Merged
ref(pallet-ud): remove params UdFirstReeval & UdReevalPeriodInBlocks
elois-ref-ud
into
master
Overview
0
Commits
2
Pipelines
0
Changes
13
Merged
Éloïs
requested to merge
elois-ref-ud
into
master
2 years ago
Overview
0
Commits
2
Pipelines
0
Changes
13
Expand
Supprime le paramètre
UdFirstReeval
pour le remplacer par un storage item qui indique la date de prochaine réévaluation.
Nécessaire pour pouvoir modifier la fréquence de réévalutation à l'avenir
Permet de ne pas avoir un paramètre qui ne sert qu'au genesis
Supprime le paramètre
UdReevalPeriodInBlocks
et change l'unité du paramètre
UdReevalPeriod
pour que ça devienne un nombre de blocs.
Évite d'avoir plusieurs paramètres différents dont l'un doit être calculé à partir des autres, donc permet de limiter les cas de mauvais paramétrage.
Implique de faire une division en plus à chaque réévaluation, mais c'est tout les 6 mois en prod donc ultra négligeable.
Edited
2 years ago
by
Éloïs
0
0
Merge request reports
Loading