CodeChain-io / CodeChain-io/foundry

snapshot_period remains unused.

Open
#191 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
36
Forks
11
PR merge metrics
No merged PRs in 30d

Description

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

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.