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

prover(conglomeration): implements the proof aggregation mechanism #564

Draft
wants to merge 57 commits into
base: prover/limitless-top-level
Choose a base branch
from

Conversation

AlexandreBelling
Copy link
Contributor

This PR implements issue(s) #

Checklist

  • I wrote new tests for my new core changes.
  • I have successfully ran tests, style checker and build against my new changes locally.
  • I have informed the team of any breaking changes if there are any.

@AlexandreBelling AlexandreBelling added the Prover Tag to use for all work impacting the prover label Jan 16, 2025
@AlexandreBelling AlexandreBelling self-assigned this Jan 16, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jan 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.49%. Comparing base (fb4ee00) to head (f4930c4).

Additional details and impacted files
@@                      Coverage Diff                      @@
##             prover/limitless-top-level     #564   +/-   ##
=============================================================
  Coverage                         68.49%   68.49%           
  Complexity                         1130     1130           
=============================================================
  Files                               321      321           
  Lines                             12842    12842           
  Branches                           1287     1287           
=============================================================
  Hits                               8796     8796           
  Misses                             3506     3506           
  Partials                            540      540           
Flag Coverage Δ *Carryforward flag
hardhat 98.74% <ø> (ø)
kotlin 66.05% <ø> (ø) Carriedforward from 2c495ad

*This pull request uses carry forward flags. Click here to find out more.

* better error message in the verifier of self-recursion
* bad condition in AllKeysInProverTranscript
* vortex proving from rs encoded matrix
* fix how the fs is extracted from the state in conglo
* make FiatShamirSetup public
* initializes the state in verifier runtime
* minor(wizard): adding GetQuery to the Runtime and GnarkRuntime interface
* fixup(conglo): support the case where there are dry rounds in vortex
* minor(mpts): using the run GetQuery method
* minor(selfrec): better error message
* test(conglo): adds a test case for arcane + lookup
@AlexandreBelling AlexandreBelling changed the title feat(conglomeration): implements the context translation prover(conglomeration): implements the proof aggregation mechanism Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Prover Tag to use for all work impacting the prover
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants