-
Notifications
You must be signed in to change notification settings - Fork 970
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
Merge discussion and todos from #2257 #2280
Comments
I would also say consider adding some fields back in despite the redundancy. Particularly, adding the block prevhash back in to the It feels like there's important simplifications that can come from making sure that there is a clean pure one-way map from |
PR addressing additional fields in |
Can we pipe becon chain block roots to eth1x as well? Would be very useful for applications that depend on beacon chain, like staking pools, which are forced to use oracle these days. |
The beacon state root would be a useful addition, not sure that individual block roots would gain anything. |
There is a plan to introduce |
(4) Addressed in #2472 |
#2257 got the basic merge spec to a solid footing. The following are the list of discussion points and todos that were not completed within the context of that PR. These are left for future work and discussion.
Note: This is not an exhaustive merge todo list but is instead only open points from #2257
application_state_transition
(should the spec know about theApplicationState
or just about its root hash)uint256
requirement on beacon-chain side inPoWBlock
and TD comparisonApplicationPayload
and where should such validation happen. At gossip, inside beacon chain, or just in application gas limit. Max TX size times max txs is 16GB!Union
type for transaction list with a singleOPAQUE_SELECTOR
for first merge fork. More specific, structured selectors for each TX type to followbase_fee
toApplicationPayload
Hash32
type to provide more specificity thanBytes32
and to further disambiguate fromRoot
Eth1Data
--DepostContractData
ApplicationPayload
to RLP blockThe text was updated successfully, but these errors were encountered: