CodeChain-io / CodeChain-io/foundry
snapshot_period remains unused.
- Vorherrschende Sprache
- Rust
- Sterne
- 36
- Forks
- 11
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
`snapshot_period` in `CommonParams` was added to determine the snapshot period. However, because of term specific data like `intermediate_reward`, the current snapshot period is fixed at 1 term.
There are two options:
1. Get rid of it from `CommonParams`
2. Modify its usage to designate the snapshot period as `snapshot_period` terms. It may accompany field renaming.
Beitragsleitfaden
Rechercherichtung
Beginne damit, CommonParams und jede Verwendung von snapshot_period zu lokalisieren, und untersuche anschließend, wie intermediate_reward und das aktuelle Snapshot-Verhalten mit einem Term behandelt werden. Bestimme, ob das beabsichtigte Ergebnis darin besteht, das Feld zu entfernen oder es so zu ändern, dass es eine Snapshot-Periode über mehrere Terme steuert, einschließlich einer möglichen Umbenennung des Feldes. Als erledigt gilt die Aufgabe, wenn das gewählte Verhalten spezifiziert und konsistent implementiert ist und die relevanten Tests aktualisiert oder hinzugefügt wurden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- rust
- Bereich
- blockchain
- Issue-Typ
- Refactoring
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 20/100