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

[EFM] Dynamic Protocol State maintains EFM by injecting EpochExtensions #5726

Closed
Tracked by #5732 ...
kc1116 opened this issue Apr 18, 2024 · 3 comments
Closed
Tracked by #5732 ...

Comments

@kc1116
Copy link
Contributor

kc1116 commented Apr 18, 2024

Context

After an initial EpochExtension is added Subsequent EpochExtensions are added to the Protocol State according to the following algorithm:

Suppose we are in EFM and are processing block B.

  • Let $\hat{V}$ be the final view of the current epoch, taking into account prior EpochExtensions.

We will add a new EpochExtension to the protocol state in block B iff both are true:
Screenshot 2024-04-18 at 2.10.13 PM.png

If we add a new EpochExtension, it will have the following form:

Screenshot 2024-04-18 at 2.10.35 PM.png

This issue is for adding EpochExtensions on an ongoing basis (maintaining EFM). #5724 is for injecting the first EpochExtension (entering EFM).

Definition of Done

Update Dynamic Protocol State Mutator to maintain EFM by periodically adding new EpochExtensions.

Further Reading

Design

@kc1116 kc1116 self-assigned this Apr 18, 2024
@kc1116 kc1116 changed the title [EFM] EpochExtension support in dynamic protocol state mutator. [EFM] EpochExtension support in dynamic protocol state mutator Apr 18, 2024
@franklywatson franklywatson added this to the EFM-Q2 milestone Apr 18, 2024
@jordanschalm jordanschalm changed the title [EFM] EpochExtension support in dynamic protocol state mutator [EFM] Dynamic Protocol State maintains EFM by injecting EpochExtensions Apr 23, 2024
@kc1116 kc1116 assigned durkmurder and unassigned kc1116 Apr 23, 2024
@AlexHentschel
Copy link
Member

see #5631 for more details

related issues:

@durkmurder
Copy link
Member

Partially implemented in #5773

@durkmurder
Copy link
Member

Remaining logic implemented in #5898

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants