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

Nullifier tree size exceeded in Sandbox after 256 blocks #2947

Closed
spalladino opened this issue Oct 20, 2023 · 0 comments · Fixed by #2988
Closed

Nullifier tree size exceeded in Sandbox after 256 blocks #2947

spalladino opened this issue Oct 20, 2023 · 0 comments · Fixed by #2988
Assignees
@rahul-kothari
Labels
T-bug Type: Bug. Something is broken. T-feedback Type: recording user feedback

Comments

@spalladino
Copy link
Collaborator

A user received the following error after having a Sandbox running for 256 blocks:
image (1)

Reported by @drgorillamd from Wonderland
@spalladino spalladino added T-bug Type: Bug. Something is broken. T-feedback Type: recording user feedback labels Oct 20, 2023
@rahul-kothari rahul-kothari added this to A3 Oct 23, 2023
@github-project-automation github-project-automation bot moved this to Todo in A3 Oct 23, 2023
@rahul-kothari rahul-kothari mentioned this issue Oct 24, 2023
Merged
rahul-kothari added a commit that referenced this issue Oct 24, 2023
@rahul-kothari
feat: change nullifier tree height to 20 (#2988)
118f9d5 
Fix #2947.

Note - had to change the nullifier height to 20 instead of 32 since
circuit tests use `NullifierMemoryTreeTestingHarness` which uses BB's
Memory Tree which is limited to a max height of 20. I can't change it to
32 due to bad_alloc.
@github-project-automation github-project-automation bot moved this from Todo to Done in A3 Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rahul-kothari rahul-kothari

Labels
T-bug Type: Bug. Something is broken. T-feedback Type: recording user feedback
Projects
A3
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: change nullifier tree height to 20 AztecProtocol/aztec-packages
2 participants
@spalladino @rahul-kothari