Skip to content
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

Implement Optimized Attestation Selection and Aggregation Strategy #9186

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

GarmashAlex
Copy link

PR Description

This PR introduces a significant functional improvement to the block production process by implementing an optimized attestation selection and aggregation strategy. The new strategy enhances validator rewards and network efficiency by:
Intelligently aggregating attestations with the same attestation data to maximize the number of unique validators included in blocks
Prioritizing attestations based on a weighted combination of:

  • Aggregation count (number of validators per attestation)
  • Inclusion delay (prioritizing older attestations that are about to expire)
  • Committee representation (ensuring diversity of committees)
    The implementation includes:
  • A new OptimizedAttestationAggregationStrategy class that handles the core logic
  • Integration with the existing BlockOperationSelectorFactory
  • Comprehensive unit tests to verify the functionality
  • Performance metrics logging to track improvements
    This optimization should lead to more efficient block space usage, increased validator participation, and potentially higher rewards for block proposers.

Documentation

  • I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

  • I thought about adding a changelog entry, and added one if I deemed necessary.

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.

1 participant