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(ivc): accumulation of step circuit instances #354

Merged

Conversation

cyphersnake
Copy link
Collaborator

Motivation
Practically complete realization of task #316

Overview
Impl of accumulation of step circuit instances

  • Addition of a step_circuit_instances_hash_accumulator to the RelaxedPlonkInstance structure.
  • Implementation of methods to fold and conditionally select step circuit instances.
  • Modification of existing methods to incorporate the new accumulator.
  • Update to testing modules to cover the changes.

@cyphersnake cyphersnake requested a review from chaosma September 25, 2024 11:03
@cyphersnake cyphersnake self-assigned this Sep 25, 2024
@cyphersnake
Copy link
Collaborator Author

cyphersnake commented Sep 25, 2024

I plan to finalize the documentation, as well as add a test to nifs mod with an instance column in the last, finalizing #316 the next PR

@cyphersnake cyphersnake force-pushed the 316-feat-instances-accumulator branch from 3dd4ffe to 698a578 Compare September 25, 2024 13:40
**Motivation**
Practically complete realization of task #316

**Overview**
Impl of accumulation of step circuit instances
@cyphersnake cyphersnake force-pushed the 316-feat-instances-accumulator-2 branch from 40789d4 to 6b0ebb0 Compare September 25, 2024 13:42
@cyphersnake cyphersnake merged commit 98f88dd into 316-feat-instances-accumulator Sep 30, 2024
1 check passed
@cyphersnake cyphersnake deleted the 316-feat-instances-accumulator-2 branch September 30, 2024 10:59
@cyphersnake cyphersnake restored the 316-feat-instances-accumulator-2 branch September 30, 2024 12:23
@cyphersnake cyphersnake deleted the 316-feat-instances-accumulator-2 branch September 30, 2024 12:23
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