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

[EIP-7549] Move committee index outside attestation #7965

Open
20 of 22 tasks
lucassaldanha opened this issue Feb 8, 2024 · 0 comments
Open
20 of 22 tasks

[EIP-7549] Move committee index outside attestation #7965

lucassaldanha opened this issue Feb 8, 2024 · 0 comments
Assignees
Labels
EIP7549 Epic Electra Issues required to implement Electra upgrade Epic

Comments

@lucassaldanha
Copy link
Member

lucassaldanha commented Feb 8, 2024

Description

Move the committee index field outside of the signed Attestation message to allow aggregation of equal consensus votes.

References

Tasks

Once the previous done, we can start merging the changes in master while progressing on the following:

Tech debt:

  • ELECTRA milestone over AggregateAttestationValidatorTest and implement ElectraAttestationValidatorTest

Possible evolutions

@lucassaldanha lucassaldanha added the Epic Electra Issues required to implement Electra upgrade label Feb 8, 2024
@mehdi-aouadi mehdi-aouadi self-assigned this Feb 24, 2024
@mehdi-aouadi mehdi-aouadi mentioned this issue Mar 6, 2024
2 tasks
@mehdi-aouadi mehdi-aouadi changed the title [EIP-7549] Move committee index outside attestation Move committee index outside attestation Mar 6, 2024
@lucassaldanha lucassaldanha changed the title Move committee index outside attestation [EIP-7549] Move committee index outside attestation Mar 11, 2024
@mehdi-aouadi mehdi-aouadi reopened this Apr 30, 2024
@tbenr tbenr self-assigned this May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EIP7549 Epic Electra Issues required to implement Electra upgrade Epic
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants