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

WIP: FP design writeup #350

Closed
wants to merge 1 commit into from
Closed

WIP: FP design writeup #350

wants to merge 1 commit into from

Conversation

puma314
Copy link
Contributor

@puma314 puma314 commented Jan 27, 2025

No description provided.

@github-actions GitHub Actions
Copy link

Metric Value
Batch Start 23,077,530
Batch End 23,077,535
Witness Generation (seconds) 60
Execution Duration (seconds) 45
Total Instruction Count 1,813,213,660
Oracle Verify Cycles 408,970,364
Derivation Cycles 920,287,897
Block Execution Cycles 325,934,862
Blob Verification Cycles 178,910,999
Total SP1 Gas 0
Number of Blocks 5
Number of Transactions 56
Ethereum Gas Used 14,059,417
Cycles per Block 362,642,732
Cycles per Transaction 32,378,815
Transactions per Block 11
Gas Used per Block 2,811,883
Gas Used per Transaction 251,061
BN Pair Cycles 0
BN Add Cycles 0
BN Mul Cycles 0
KZG Eval Cycles 0
EC Recover Cycles 620,004
P256 Verify Cycles 0

fakedev9999
fakedev9999 reviewed Jan 27, 2025
View reviewed changes
book/fault_proofs/fault_proof_architecture.md

Initializes the dispute game with:

- Parent game reference (if any)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Except the very first game, there'll be a parent game right?

book/fault_proofs/fault_proof_architecture.md
Comment on lines +128 to +129
2. **Challenge Bond**: Required to challenge a proposal
3. **Proof Reward**: Paid to successful provers
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aren't they the same thing?

@fakedev9999
Copy link
Member

Cherry-picked and updated in #342.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fakedev9999 fakedev9999 fakedev9999 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@puma314 @fakedev9999