-
Notifications
You must be signed in to change notification settings - Fork 703
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
Ignore aggregate subsets on gossip #3238
Labels
Comments
bors bot
pushed a commit
that referenced
this issue
Jun 27, 2023
## Issue Addressed Resolves #3238 ## Proposed Changes Please list or describe the changes introduced by this PR. ## Additional Info Please provide any additional information. For example, future considerations or information useful for reviewers.
Completed in #3493 |
ghost
pushed a commit
to oone-world/lighthouse
that referenced
this issue
Jul 13, 2023
## Issue Addressed Resolves sigp#3238 ## Proposed Changes Please list or describe the changes introduced by this PR. ## Additional Info Please provide any additional information. For example, future considerations or information useful for reviewers.
Woodpile37
pushed a commit
to Woodpile37/lighthouse
that referenced
this issue
Jan 6, 2024
Resolves sigp#3238 Please list or describe the changes introduced by this PR. Please provide any additional information. For example, future considerations or information useful for reviewers.
Woodpile37
pushed a commit
to Woodpile37/lighthouse
that referenced
this issue
Jan 6, 2024
Resolves sigp#3238 Please list or describe the changes introduced by this PR. Please provide any additional information. For example, future considerations or information useful for reviewers.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
We should implement: ethereum/consensus-specs#2847
This should help filter out duplicates and subset aggregate attestations.
We might need another error case for attestation and sync committee aggregate processing. Potentially @michaelsproul may be best suited for that modification.
We should also adjust the scoring parameters on these topics to account of the lower expected number of messages. Lodestar have indicated they see ~70-80% reduction in message count.
The text was updated successfully, but these errors were encountered: