-
Notifications
You must be signed in to change notification settings - Fork 139
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
core: store StateLeaf
objects in IncrementalQuinTree
#917
Comments
Given some thoughts on this and I think it might be counterproductive to do that, as we use the |
I see |
I don't think I properly understood then? you mentioned that we should store the |
@baumstern wondering if you had a look at my latest reply on this |
Sorry, I wrote a reply but didn't posted it 🤦 so..currently, the |
My bad now for a late reply! So the difference would just be that the arrays instead of being in the |
Context
IncrementalQuinTree
currently holds onlyStateLeaf
hashes. Suggesting an enhancement to storeStateLeaf
objects directly, removing the need for thestateLeaves
array in MaciState and in Poll object.Objective
Refactor
IncrementalQuinTree
to encapsulateStateLeaf
objects, simplifying the state management withinMaciState
.The text was updated successfully, but these errors were encountered: