Skip to content

Only add proposer boost to same shufflings as head#16846

Merged
syjn99 merged 6 commits into
developfrom
dependent_root_pb
Jun 8, 2026
Merged

Only add proposer boost to same shufflings as head#16846
syjn99 merged 6 commits into
developfrom
dependent_root_pb

Conversation

@potuz

@potuz potuz commented May 27, 2026

Copy link
Copy Markdown
Contributor

When an incoming block arrives, only add proposer boost to it if the proposer is the same as the expected one from the current head's shuffling perspective. Head is cached before insertion and we do not use the shuffling that would be expected after counting the attestations in the block for example,

potuz added 2 commits May 26, 2026 21:57
When an incoming block arrives, only add proposer boost to it if the
proposer is the same as the expected one from the current head's
shuffling perspective. Head is cached before insertion and we do not use
the shuffling that would be expected after counting the attestations in
the block for example.
s.proposerBoostRoot = root
depEpoch := slots.ToEpoch(currentSlot)
if depEpoch > 1 {
depEpoch -= 2

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

block epoch is 100, wouldnt you get epoch 97's root but you want epoch 98's root right?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed in 6aaebba

@potuz potuz enabled auto-merge May 29, 2026 14:54
@potuz potuz disabled auto-merge June 3, 2026 10:11
@potuz potuz enabled auto-merge June 3, 2026 10:11
@barnabasbusa

Copy link
Copy Markdown
Contributor

ship it

@potuz potuz added this pull request to the merge queue Jun 8, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 8, 2026
@syjn99 syjn99 added this pull request to the merge queue Jun 8, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jun 8, 2026
@syjn99 syjn99 added this pull request to the merge queue Jun 8, 2026
Merged via the queue into develop with commit 353d74c Jun 8, 2026
31 checks passed
@syjn99 syjn99 deleted the dependent_root_pb branch June 8, 2026 07:14
james-prysm pushed a commit that referenced this pull request Jun 8, 2026
When an incoming block arrives, only add proposer boost to it if the
proposer is the same as the expected one from the current head's
shuffling perspective. Head is cached before insertion and we do not use
the shuffling that would be expected after counting the attestations in
the block for example,
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.

5 participants