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.

Sign up for GitHub

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

Jump to bottom

Update AggregationMerkleSumTree #5

Closed
enricobottazzi opened this issue Nov 27, 2023 · 1 comment
Closed

Update AggregationMerkleSumTree #5

enricobottazzi opened this issue Nov 27, 2023 · 1 comment
Assignees
@enricobottazzi

Comments

@enricobottazzi
Copy link
Member

According to the new modification on the Tree trait performed in summa-dev/summa-solvency#192 and summa-dev/summa-solvency#188, the AggregationMerkleSumTree (https://github.com/summa-dev/summa-aggregation/blob/main/src/aggregation_merkle_sum_tree.rs#L16-L21) needs to be updated to be compatible with the trait
@enricobottazzi enricobottazzi self-assigned this Nov 27, 2023
@enricobottazzi enricobottazzi transferred this issue from summa-dev/summa-solvency Dec 1, 2023
@sifnoc
Copy link
Member

sifnoc commented Dec 13, 2023

Updated at #3

@sifnoc sifnoc closed this as completed Dec 13, 2023
@github-project-automation github-project-automation bot moved this from Backlog to Merged in ∑ Summa Dec 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@enricobottazzi enricobottazzi

Labels
None yet
Projects
∑ Summa
Status: Merged
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sifnoc @enricobottazzi