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

o1vm/lookup: add lookup prover until alpha #3000

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

marcbeunardeau88
Copy link
Contributor

first part of the prover that will prove that the sum of the lookup from one circuit goes from one public to one public output

@marcbeunardeau88 marcbeunardeau88 force-pushed the marc/auxiliary_prover_rebased branch 2 times, most recently from 3af7e02 to 039d27d Compare February 5, 2025 13:53
@marcbeunardeau88 marcbeunardeau88 force-pushed the marc/auxiliary_prover_rebased branch from 039d27d to d41ef48 Compare February 5, 2025 14:07
Copy link

codecov bot commented Feb 5, 2025

Codecov Report

Attention: Patch coverage is 0% with 85 lines in your changes missing coverage. Please review.

Project coverage is 76.69%. Comparing base (84a357a) to head (d41ef48).
Report is 30 commits behind head on master.

Files with missing lines Patch % Lines
o1vm/src/pickles/lookup_prover.rs 0.00% 85 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3000      +/-   ##
==========================================
- Coverage   77.15%   76.69%   -0.46%     
==========================================
  Files         260      262       +2     
  Lines       61529    61967     +438     
==========================================
+ Hits        47470    47526      +56     
- Misses      14059    14441     +382     

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

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