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

Stark: Prover and Verifier over field extensions v3 #724

Merged
merged 49 commits into from
Dec 27, 2023

Conversation

schouhy
Copy link
Contributor

@schouhy schouhy commented Dec 18, 2023

Prover and Verifier over field extensions v3

Description

This is the last PR in the series to refactor the Prover and Verifier to use field extensions. This PR modifies the tables and frames to be aware of different fields coming from evaluations of main (base field) and auxiliary trace (field extension) columns. This PR includes:

  • Implementation of EvaluationTable struct that can hold main and auxiliary evaluation tables that take values in different fields.
  • Modification of the Frame struct to hold main and auxiliary evaluations in the same spirit.
  • Modification of the AIR api to be able to evaluate constraints using values in different fields.
  • Adapt existing AIR implementations to the above changes.

@codecov-commenter
Copy link

Codecov Report

Attention: 10 lines in your changes are missing coverage. Please review.

Comparison is base (e1a8716) 95.66% compared to head (9779657) 95.63%.

Files Patch % Lines
provers/stark/src/trace.rs 83.33% 7 Missing ⚠️
provers/stark/src/constraints/boundary.rs 92.85% 2 Missing ⚠️
provers/stark/src/table.rs 98.61% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #724      +/-   ##
==========================================
- Coverage   95.66%   95.63%   -0.04%     
==========================================
  Files         136      136              
  Lines       30672    30882     +210     
==========================================
+ Hits        29342    29533     +191     
- Misses       1330     1349      +19     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@schouhy schouhy marked this pull request as ready for review December 20, 2023 21:26
@schouhy schouhy requested review from ajgara and a team as code owners December 20, 2023 21:26
@MauroToscano
Copy link
Collaborator

Round 2 has a slowdown of around 2%

  • Started round 2: Compute composition polynomial
    Time spent: 18.943324653s
  • Started round 2: Compute composition polynomial
    Time spent: 12.936840265s

We should explore this a bit

@MauroToscano
Copy link
Collaborator

Round 2 has a slowdown of around 2%

* Started round 2: Compute composition polynomial
  Time spent: 18.943324653s

* Started round 2: Compute composition polynomial
  Time spent: 12.936840265s

We should explore this a bit

After double checking, it seems this version is faster. Incredible PR

@entropidelic entropidelic added this pull request to the merge queue Dec 27, 2023
Merged via the queue into main with commit c0e190f Dec 27, 2023
7 checks passed
@entropidelic entropidelic deleted the stark-prover-over-field-extensions-v3 branch December 27, 2023 19:32
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.

5 participants