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.

Sign up for GitHub

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

Jump to bottom

Impl chainstate root computation #130

Closed
Tracked by #135
, #150
...
delbonis opened this issue Jul 15, 2024 · 0 comments · Fixed by #171
Closed
Tracked by #135
, #150
...

Impl chainstate root computation #130

delbonis opened this issue Jul 15, 2024 · 0 comments · Fixed by #171
Assignees
@prajwolrg
Labels
consensus Consensus-critical, high-sensitivity required enhancement New feature or request

Comments

@delbonis
Copy link
Contributor

We need to compute state roots for the chainstate so we have something to fill in in the state_root field in the header and can verify that someone else's state is the same as ours.

This doesn't have to be overly sophisticated, since we may swap out our internal type definitions with SSZ type definitions, which defines all of this more rigorously.
This was referenced Jul 15, 2024
Closed
Closed
@delbonis delbonis added enhancement New feature or request consensus Consensus-critical, high-sensitivity required labels Jul 15, 2024
@bewakes bewakes assigned MdTeach and prajwolrg and unassigned MdTeach Jul 17, 2024
@prajwolrg prajwolrg mentioned this issue Jul 24, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@prajwolrg prajwolrg

Labels
consensus Consensus-critical, high-sensitivity required enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ChainState: Calculate state root alpenlabs/strata
3 participants
@prajwolrg @MdTeach @delbonis