-
Notifications
You must be signed in to change notification settings - Fork 317
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
feat: Proving the rollup circuits #5599
Merged
Merged
Changes from 24 commits
Commits
Show all changes
50 commits
Select commit
Hold shift + click to select a range
e38f055
WIP
PhilWindle 0abfe1c
Merge branch 'master' into pw/generate-keys
PhilWindle 78b36ca
Merge fix
PhilWindle 0a07ffb
WIP
PhilWindle 029912a
WIP
PhilWindle 04e5bcb
WIP
PhilWindle dce3bbf
Merge branch 'master' into pw/generate-keys
PhilWindle 43022c1
WIP
PhilWindle c50d853
Merge branch 'master' into pw/generate-keys
PhilWindle e16c245
WIP
PhilWindle 41cd872
WIP
PhilWindle 5050114
WIP
PhilWindle 86fd349
Merge branch 'master' into pw/generate-keys
PhilWindle 64e6a39
WIP
PhilWindle 660a21b
Fixes
PhilWindle d0af6b7
Fixes
PhilWindle 5bb7de2
Cleanup
PhilWindle 529ec78
Merge branch 'master' into pw/generate-keys
PhilWindle 18b93b5
Revert bb changes
PhilWindle 2966163
Revert further bb changes
PhilWindle cb54097
Some more cleanup
PhilWindle 2352c24
Further cleanup
PhilWindle e0b33ce
Comments and cleanup
PhilWindle ff861d1
Fixes
PhilWindle 4d263c8
Test fixes
PhilWindle 2ea7278
Cleanup and breaking up test files
PhilWindle 1ba8845
Formatting
PhilWindle 3533c3a
Review fixes
PhilWindle ef6f80f
Review fixes
PhilWindle 868a94f
Review changes
PhilWindle a6986f5
Reverted breakage
PhilWindle 80aa850
Merge branch 'pw/generate-keys' of github.com:AztecProtocol/aztec-pac…
PhilWindle eba7378
Verify proof in test
PhilWindle 9b07deb
Verify proof in test
PhilWindle ed3db50
Merge branch 'pw/generate-keys' of github.com:AztecProtocol/aztec-pac…
PhilWindle 1a0bc82
Merge branch 'master' into pw/generate-keys
PhilWindle 1d2cd6c
Merge fixes
PhilWindle 82035e3
Merge fixes
PhilWindle cfc6ea1
Copy the ACVM in test dockerfile
PhilWindle cfeb8e1
Fixed dockerfile
PhilWindle 6bb1942
Increased test timeouts
PhilWindle 3838a0c
Test fixes
PhilWindle 6080635
Debugging CI
PhilWindle e6a9bd3
Move prover client testing to it's own job
PhilWindle 960b709
Fixes
PhilWindle 0a049bb
Further debugging
PhilWindle dcf6755
More debugging
PhilWindle 98755f4
Merge branch 'master' into pw/generate-keys
PhilWindle 460c4e4
Update build manifest dependencies
PhilWindle f28abe8
Cleanup
PhilWindle File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -18,6 +18,7 @@ | |
"bbfree", | ||
"bbmalloc", | ||
"benesjan", | ||
"Bincode", | ||
"bleurgh", | ||
"bodyparser", | ||
"bootnode", | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Pedantic ts comment: I'd rather use a discriminated union here, so the compiler can check eg that when
type = TAIL
theninputs
must bePublicKernelTailCircuitPrivateInputs
and notPublicKernelCircuitPrivateInputs
.So the above would become:
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.
Hah, just saw you did this already with the BBResult type!
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.
I agree, that's much better.