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

fix: fetch randomness from chain for batch verification #812

Merged
merged 1 commit into from
Mar 28, 2025

Conversation

th7nder
Copy link
Contributor

@th7nder th7nder commented Mar 20, 2025

Description

Now the randomness if fetch via the randomness pallet and used to seed the rng to generate random numbers for Groth16 batch verification.

Closes #810.

Checklist

  • Make sure that you described what this change does.
  • Have you tested this solution?

@th7nder th7nder added the enhancement New feature or request label Mar 20, 2025
@th7nder th7nder added this to the Phase 3 milestone Mar 20, 2025
@th7nder th7nder self-assigned this Mar 20, 2025
@th7nder th7nder added the ready for review Review is needed label Mar 20, 2025
@th7nder th7nder force-pushed the fix/810/groth16-randomness branch from bd47275 to 1601148 Compare March 20, 2025 14:00
@th7nder th7nder added ready for review Review is needed and removed ready for review Review is needed labels Mar 20, 2025
@th7nder th7nder requested a review from jmg-duarte March 20, 2025 14:00
@th7nder th7nder mentioned this pull request Mar 20, 2025
55 tasks
Copy link
Contributor

@jmg-duarte jmg-duarte left a comment

Choose a reason for hiding this comment

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

LGTM

@th7nder th7nder enabled auto-merge (squash) March 28, 2025 12:35
@th7nder th7nder merged commit 9721630 into develop Mar 28, 2025
6 checks passed
@th7nder th7nder deleted the fix/810/groth16-randomness branch March 28, 2025 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ready for review Review is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fetch randomness from the chain for batch Groth16 verify
3 participants