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.

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: use deployed kzg-rs #76

Merged
merged 5 commits into from
Aug 27, 2024
Merged

feat: use deployed kzg-rs #76

merged 5 commits into from
Aug 27, 2024

Conversation

0xWOLAND
Copy link
Contributor

@0xWOLAND 0xWOLAND commented Aug 27, 2024

  • Use stable release of kzg-rs.
  • Batch KZG verification.

@0xWOLAND 0xWOLAND requested a review from ratankaliani August 27, 2024 18:45
@ratankaliani
Copy link
Member

Add the batch KZG verification to

let kzg_settings = get_kzg_settings();
for (commitment, blob) in blobs.iter() {
println!("cycle-tracker-report-start: blob-verification");
kzg_rs::KzgProof::verify_blob_kzg_proof(
KzgRsBlob::from_slice(&blob.data.0).unwrap(),
&Bytes48::from_slice(&commitment.0).unwrap(),
&Bytes48::from_slice(&blob.kzg_proof.0).unwrap(),
&kzg_settings,
)
.map_err(|e| anyhow!("blob verification failed: {:?}", e))?;
println!("cycle-tracker-report-end: blob-verification");
}

@0xWOLAND 0xWOLAND changed the base branch from ratan/kzg-blob-verification to main August 27, 2024 18:55
@0xWOLAND 0xWOLAND force-pushed the bhargav/kzg-rs-dep branch from 8ad71e4 to 09303e2 Compare August 27, 2024 18:59
Copy link
Member

@ratankaliani ratankaliani left a comment

Choose a reason for hiding this comment

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

LGTM

@ratankaliani ratankaliani merged commit 06ec2f3 into main Aug 27, 2024
1 check passed
@ratankaliani ratankaliani deleted the bhargav/kzg-rs-dep branch September 24, 2024 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants