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

New Trie Research #6558

Open
Tracked by #6577 ...
j1010001 opened this issue Oct 10, 2024 · 1 comment
Open
Tracked by #6577 ...

New Trie Research #6558

j1010001 opened this issue Oct 10, 2024 · 1 comment
Assignees
Labels
Execution Cadence Execution Team

Comments

@j1010001
Copy link
Member

Why (Objective)

Long-term: Execution state scalability (state proofs)

How will we measure success (Key Results) ?

6-month goal: Complete Architecture & Design so we can start implementation.

Effort Estimate

Time-box 40% of @fxamacker's capacity for this cycle for 2 days/week

DACI

Role Assigned
Driver @AlexHentschel
Approver @dete
Consulted @dete, @fxamacker, @ramtinms
Informed Flow Protocol engineering
@j1010001 j1010001 added the Execution Cadence Execution Team label Oct 10, 2024
@j1010001 j1010001 changed the title [PLACEHOLDER] New Trie Research New Trie Research Oct 10, 2024
@bluesign
Copy link
Contributor

( Disclaimer: Noob comment, just trying to find out what I am missing here )

I commented on this long before somewhere, but couldn't find the comment yet. But basically it was something like this:

I checked a bit and current mtrie is pretty good actually, why research a new one ? I think PathFinder is the problem here;

if we had a PathFinder function F; something like accountIndex(owner) || [ ( slabId | ^specialRegisterIndex ) wouldn't it be enough ?

accountIndex is bound to 45 bits, if we reserve 64 bits for future. 192 bits for registers are more than enough. ( currently it is limited to 64 bits iirc )

this will compress the mtrie much better ( on accountIndex and slabId ) also it lets proofs to share most of the paths.

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

No branches or pull requests

4 participants