CodeChain-io / CodeChain-io/foundry
snapshot_period remains unused.
オープン
- 主要言語
- 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