Skip to content

Fix attestation inclusion delay calculation#45

Merged
JamesEjembi merged 4 commits into
VeriNode-Labs:mainfrom
OkeyAmy:fix-attestation-inclusion-delay-calculation
Jun 24, 2026
Merged

Fix attestation inclusion delay calculation#45
JamesEjembi merged 4 commits into
VeriNode-Labs:mainfrom
OkeyAmy:fix-attestation-inclusion-delay-calculation

Conversation

@OkeyAmy

@OkeyAmy OkeyAmy commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Closes #14

Summary

  • Add attestation inclusion-delay accounting that measures wall-clock slots directly, including skipped slots.
  • Add linear inclusion-delay reward calculation with zero reward at delay 32.
  • Add consensus branch weighting that consumes wall-clock inclusion-delay rewards.
  • Add regression coverage for skipped slots and fork-choice weight invariance across skipped-slot distributions.

Testing

  • cargo build --verbose
  • cargo test --verbose

@JamesEjembi

Copy link
Copy Markdown
Contributor

ci failed

@JamesEjembi JamesEjembi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@JamesEjembi JamesEjembi merged commit 0f82520 into VeriNode-Labs:main Jun 24, 2026
1 check passed
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.

Attestation Inclusion Delay Tracking Under-Counted During Skipped Slots

2 participants