Skip to content

Rough prototype sending fcU earlier #7752

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: unstable
Choose a base branch
from

Conversation

michaelsproul
Copy link
Member

Issue Addressed

This is option 2 from:

Proposed Changes

  • Call the full recompute_head* routine in import_block so that fcU gets sent to the EL prior to the block being fully imported.
  • To work around the assumptions of recompute_head (head block and state exist in store): pass the beacon block and state to the function.

Additional Info

There are multiple problems with the current implementation but I'm curious to see how much this basic approach nets us before going further trying to fix the issues. Incomplete list of current flaws:

  • I broke the attester cache's handling of blobs and columns (easy to fix)
  • TODO

@michaelsproul michaelsproul added optimization Something to make Lighthouse run more efficiently. builder API labels Jul 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
builder API optimization Something to make Lighthouse run more efficiently.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant