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: proof generation using local srs transcript file #5

Open
wants to merge 3 commits into
base: latest
Choose a base branch
from

Conversation

michaelelliot
Copy link

Implemented additional functions for proof generation, offering the choice between using a local SRS transcript file or fetching from a default or specified remote URL.

Refactored existing prove function to use a new shared function prove_with_srs, which accepts an SRS strategy object for SRS data handling.

Implemented support for loading local SRS data in the introduced LocalSrs trait, which reads the necessary SRS data from a given file path, and updated NetSrs trait to optionally accept a custom URL. This makes the proof generation process more portable and less reliant on an internet connection.

Added serial test annotations to prevent state pollution between tests that access shared resources.

Implemented additional functions for proof generation leveraging either local or remote SRS data, offering the choice between using a local SRS transcript file or fetching from a default or specified remote URL.

Refactored existing `prove` function to use a new shared utility function `prove_with_srs`, which accepts an SRS strategy object for SRS data handling.

Implemented support for loading local SRS data in the introduced `LocalSrs` trait, which reads the necessary SRS data from a given file path, and updated `NetSrs` trait to optionally accept a custom URL. This makes the proof generation process more portable and less reliant on an internet connection.

Added serial test annotations to prevent state pollution between tests that access shared resources.
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.

1 participant