Skip to content

kip-227: fill CandTesting at epoch start blocks#117

Open
ian0371 wants to merge 3 commits intomainfrom
kip-227-epoch
Open

kip-227: fill CandTesting at epoch start blocks#117
ian0371 wants to merge 3 commits intomainfrom
kip-227-epoch

Conversation

@ian0371
Copy link
Copy Markdown
Contributor

@ian0371 ian0371 commented May 8, 2026

Proposed changes

Fill CandTesting to header.VRank at epoch start blocks

Types of changes

  • Bugfix
  • KIP Proposal
  • KIP Improvement

Checklist

  • Used the suggested template: https://github.com/kaiachain/KIPs/blob/main/kip-template.md
  • I have read the CLA and signed by comment I have read the CLA Document and I hereby sign the CLA in first time contribution
  • I have added necessary documentation (if appropriate)
  • Any dependent changes have been merged and published in downstream modules

Related issues

#27

Further comments

@ian0371 ian0371 requested a review from hyunsooda May 8, 2026 00:23
@ian0371 ian0371 self-assigned this May 8, 2026
Copy link
Copy Markdown
Contributor

@hyunsooda hyunsooda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Currently the usage of VRank looks scattered throuight the specification. To clarify, how do you think to add "VRank encodes different payloads depending on block position" with the two category: non-epoch and epoch

For example,

┌───────────────────────┬─────────────────────────────────────────────┐
│    Block position     │                header.VRank                 │
├───────────────────────┼─────────────────────────────────────────────┤
│ N % EPOCH_LENGTH != 0 │ RLPEncode(cfReport(N)), or nil if empty     │
├───────────────────────┼─────────────────────────────────────────────┤
│ N % EPOCH_LENGTH == 0 │ RLPEncode(CandTesting(N)) (MUST NOT be nil) │
└───────────────────────┴─────────────────────────────────────────────┘

@ian0371
Copy link
Copy Markdown
Contributor Author

ian0371 commented May 8, 2026

@hyunsooda updated. PTAL

@ian0371 ian0371 requested a review from hyunsooda May 8, 2026 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants