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

feat: add method to compute storage root #1243

Closed
obatirou opened this issue Jun 28, 2024 · 1 comment
Closed

feat: add method to compute storage root #1243

obatirou opened this issue Jun 28, 2024 · 1 comment

Comments

@obatirou
Copy link
Collaborator

Feature Request

Describe the Feature Request

Add a method root to compute the storage root.
Necessary step before #1234.

For reference: https://github.com/ethereum/execution-specs/blob/master/src/ethereum/shanghai/trie.py#L89
List of function to replicate:

  1. _prepare_dict
  2. patricialize
  3. encode_internal_node
  4. nibble_list_to_compact
  5. bytes_to_nibble_list

encode_node is not needed for storage as it is bytes
@obatirou obatirou self-assigned this Jun 28, 2024
@ClementWalter ClementWalter assigned enitrat and unassigned obatirou Jul 9, 2024
@enitrat enitrat mentioned this issue Jul 22, 2024
Closed
7 tasks
@Eikix Eikix moved this from 🆕 Backlog to 🏗 In progress in Kakarot's Project Management Jul 29, 2024
@Eikix Eikix moved this from 🏗 In progress to 🔖 Ready in Kakarot's Project Management Aug 9, 2024
@ClementWalter
Copy link
Member

closed in favor of the work on keth

@github-project-automation github-project-automation bot moved this from 🆕 Backlog to ✅ Done in Kakarot's Project Management Oct 9, 2024
matthieuauger pushed a commit to matthieuauger/kakarot that referenced this issue Nov 9, 2024
@tcoratger
Add unit tests for EthereumTransactionStore (kkrt-labs#1243)
ec99026 
* Add unit tests for EthereumTransactionStore

* fix StoredTransactionHash

* fix comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
Kakarot's Project Management
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: compute storage root kkrt-labs/kakarot
3 participants
@ClementWalter @enitrat @obatirou