-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Fork choice spec checklist #1053
Comments
This also needs to be added. In the context of a non-executable spec, it is a one-liner |
We also need to be mindful of the DoS where many forks need to be followed through epoch transitions. |
FWIW my current code ( https://github.com/ethereum/eth2.0-specs/pull/1068/files#diff-65ff52fa41121e465ff23d3b53fd0518R205 ) de facto makes the winner be "the first finalized head you discovered", ie. it's a memory-ful fork choice rule. And it has to be memory-ful because that's how weak subjectivity works anyway.....
I have been waiting for Yan to finish a liveness proof to finalize this. |
all items except (2) are addressed. Closing in favor of #1520 |
state. latest_eth1_data
to be valid and available.state.fork
.Eth2Genesis
is removed from the deposit contract (see CHAIN_START_FULL_DEPOSIT_THRESHOLD bypass #1048). Formalise the genesis event in the fork choice rule.SECONDS_PER_SLOT
: Potentially increase to 8 or 12 seconds.finalized_head
be the finalized block with the highest epoch." is not well-defined if there are two finalized heads with the highest epoch.The text was updated successfully, but these errors were encountered: