CodeChain-io / CodeChain-io/foundry

snapshot_period remains unused.

Offen
#191 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.