-
Notifications
You must be signed in to change notification settings - Fork 3
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
Optimize state sync #5
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues were found. Need to increase minor version (-> 0.3.10) for libzkbob-rs and libzkbob-rs-wasm
Why we need to insert new commitments each time after notes or an account is decrypted? libzkbob-rs/libzkbob-rs-wasm/src/client/mod.rs Lines 304 to 307 in 2537c16
libzkbob-rs/libzkbob-rs-wasm/src/client/mod.rs Lines 353 to 356 in 2537c16
Doesn't this part cover this case? libzkbob-rs/libzkbob-rs-wasm/src/client/mod.rs Lines 382 to 385 in 2537c16
|
It is necessary because we need to have correct merkle tree state before calling add_full_tx. It makes sense to update merkle tree only once with tx commitments of others' transactions and leafs of our transactions. But such approach requires rethinking of the whole process of merkle tree updating, so I have decided to leave it to upcoming updates. |
No description provided.