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

Shorten wallet tree #446

Merged
merged 22 commits into from Sep 17, 2019

Conversation

@willmeister
Copy link
Collaborator

commented Sep 17, 2019

Description

  • Shortens the rollup tree to max height 32 levels
  • Adds address-to-tree-key map and key creation
  • Adds simple caching in the Aggregator

Metadata

Fixes

  • [Sort of] Fixes #443

Contributing Agreement

willmeister added 22 commits Sep 9, 2019
…ate machine during async read-modify-write flows
… Also added conecpt of a RollupBlock
- Solidifying rollup block and transition concepts
- Stubbing out block submission
- Guaranteeing transitions are in order with locks

Merge branch 'master' into feat/437/PersistPendingBlock
…ex, and start/end root hash
- Removing the "Mock" label on wallet classes
- Making it so the client handles communication but passes through responses so that wallet can check signatures and handle responses as it sees fit
…Machine tree can handle
…d max in RollupStateMachine
Copy link
Contributor

left a comment

Alright this looks pretty good / straightforward! Appppprooooooving!

@willmeister willmeister merged commit 590f32d into plasma-group:master Sep 17, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@willmeister willmeister deleted the willmeister:shorten_wallet_tree branch Sep 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.