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

Don't resolve post values #531

Open
gballet opened this issue Jan 7, 2025 · 0 comments
Open

Don't resolve post values #531

gballet opened this issue Jan 7, 2025 · 0 comments

Comments

@gballet
Copy link
Owner

gballet commented Jan 7, 2025

As of #527, pre-values are no longer resolved. I thought it would be easy to do the same thing for post values, but not so: calling IntermediateRoot will both update the tree and hash it, which means that the tree will be collapsed. For this purpose, the proof-passing needs to happen in between the tree finalization and its hashing. This means an interface change.

We could do this right away, but it might be redundant with the MPT witness stuff. I will therefore explore this avenue before implementing it myself. This is an issue to remind myself to revisit this later.

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

No branches or pull requests

1 participant