CodeChain-io / CodeChain-io/foundry

snapshot_period remains unused.

オープン
#191 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Rust
スター
36
フォーク
11
PR マージ指標
30日以内にマージされた PR はありません

説明

`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.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

まず CommonParams と snapshot_period のすべての使用箇所を特定し、次に intermediate_reward と現在の単一項の snapshot 動作がどのように扱われているかを調べます。意図された結果がフィールドの削除なのか、それとも複数項の snapshot 期間を制御するようにすることなのかを判断し、フィールド名の変更も含めて検討します。選択した動作が仕様化され、一貫して実装され、関連するテストが更新または追加されていれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
rust
領域
blockchain
issue の種類
リファクタリング
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
20/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。