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

feat(taiko-client): submit batch proofs #18163

Draft
wants to merge 47 commits into
base: main
Choose a base branch
from
Draft

feat(taiko-client): submit batch proofs #18163

wants to merge 47 commits into from

Conversation

YoGhurt111
Copy link
Contributor

No description provided.

@YoGhurt111 YoGhurt111 self-assigned this Sep 23, 2024
@YoGhurt111 YoGhurt111 marked this pull request as draft September 23, 2024 09:13
@openzeppelin-code OpenZeppelin Code
Copy link

openzeppelin-code bot commented Sep 23, 2024 •
edited
Loading

feat(taiko-client): submit batch proofs

Generated at commit: fd5c6ee8966005001f174c7c94335c5dc712aa80

🚨 Report Summary

Severity Level Results
Contracts Critical
High
Medium
Low
Note
Total		 3
3
0
7
43
56
Dependencies Critical
High
Medium
Low
Note
Total		 0
0
0
0
0
0

For more details view the full report in OpenZeppelin Code Inspector

davidtaikocha
davidtaikocha reviewed Sep 24, 2024
View reviewed changes
packages/taiko-client/cmd/flags/prover.go Outdated Show resolved Hide resolved
packages/taiko-client/cmd/flags/prover.go Outdated Show resolved Hide resolved
packages/taiko-client/cmd/flags/prover.go Outdated Show resolved Hide resolved
davidtaikocha
davidtaikocha reviewed Sep 24, 2024
View reviewed changes
packages/taiko-client/prover/proof_producer/dummy_producer.go Show resolved Hide resolved
packages/taiko-client/prover/proof_producer/dummy_producer.go Show resolved Hide resolved
packages/taiko-client/prover/proof_producer/sgx_producer.go Outdated Show resolved Hide resolved
packages/taiko-client/prover/proof_producer/sgx_producer.go Outdated Show resolved Hide resolved
packages/taiko-client/prover/proof_producer/sgx_producer.go Show resolved Hide resolved
packages/taiko-client/prover/proof_producer/sgx_producer.go Outdated Show resolved Hide resolved
packages/taiko-client/prover/proof_producer/sgx_producer.go Outdated Show resolved Hide resolved
packages/taiko-client/prover/proof_submitter/proof_buffer.go Outdated Show resolved Hide resolved
packages/taiko-client/prover/proof_submitter/proof_buffer.go Show resolved Hide resolved
packages/taiko-client/prover/proof_submitter/proof_submitter.go Outdated Show resolved Hide resolved
YoGhurt111 and others added 11 commits September 24, 2024 16:14
@YoGhurt111 @davidtaikocha
Update packages/taiko-client/cmd/flags/prover.go
1f46ea9 
Co-authored-by: David <[email protected]>
@YoGhurt111 @davidtaikocha
Update packages/taiko-client/cmd/flags/prover.go
64b9de1 
Co-authored-by: David <[email protected]>
@YoGhurt111 @davidtaikocha
Update packages/taiko-client/prover/proof_producer/sgx_producer.go
66a3efb 
Co-authored-by: David <[email protected]>
@YoGhurt111 @davidtaikocha
Update packages/taiko-client/prover/proof_submitter/proof_submitter.go
d1d2d55 
Co-authored-by: David <[email protected]>
@YoGhurt111 @davidtaikocha
Update packages/taiko-client/cmd/flags/prover.go
88d7a43 
Co-authored-by: David <[email protected]>
@YoGhurt111 @davidtaikocha
Update packages/taiko-client/prover/proof_producer/sgx_producer.go
b5c5968 
Co-authored-by: David <[email protected]>
@YoGhurt111 @davidtaikocha
Update packages/taiko-client/prover/proof_producer/sgx_producer.go
236e660 
Co-authored-by: David <[email protected]>
@YoGhurt111 @davidtaikocha
Update packages/taiko-client/prover/proof_producer/sgx_producer.go
24add9c 
Co-authored-by: David <[email protected]>
@YoGhurt111
Merge branch 'main' into feature/batch-proof
dd4e626
@YoGhurt111
changes based on comments
e20e064
@YoGhurt111
Merge branches 'feature/batch-proof' and 'feature/batch-proof' of git…
948c55d 
…hub.com:taikoxyz/taiko-mono into feature/batch-proof
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mask-pp mask-pp mask-pp left review comments

@davidtaikocha davidtaikocha davidtaikocha left review comments

@smtmfft smtmfft Awaiting requested review from smtmfft

At least 2 approving reviews are required to merge this pull request.

Assignees

@YoGhurt111 YoGhurt111

Labels
area.client category.feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@YoGhurt111 @mask-pp @davidtaikocha