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

Refactor AIR #937

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

Refactor AIR #937

wants to merge 3 commits into from

Conversation

ColoCarletti
Copy link
Member

Refactor AIR

This PR resumes the changes made for AIR in the following PRs: 834 and 697.

Description of the pull request changes and motivation.

Type of change

Please delete options that are not relevant.

  • Refactor

Checklist

  • Linked to Github Issue
  • Unit tests added
  • This change requires new documentation.
    • Documentation has been added/updated.
  • This change is an Optimization
    • Benchmarks added/run

@ColoCarletti ColoCarletti marked this pull request as ready for review October 31, 2024 14:15
@ColoCarletti ColoCarletti requested a review from a team as a code owner October 31, 2024 14:15
@codecov-commenter
Copy link

codecov-commenter commented Oct 31, 2024

Codecov Report

Attention: Patch coverage is 81.81818% with 52 lines in your changes missing coverage. Please review.

Project coverage is 70.19%. Comparing base (8dfddac) to head (591deff).

Files with missing lines Patch % Lines
provers/stark/src/trace.rs 59.59% 40 Missing ⚠️
provers/stark/src/table.rs 44.44% 5 Missing ⚠️
provers/stark/src/traits.rs 55.55% 4 Missing ⚠️
provers/stark/src/prover.rs 98.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #937      +/-   ##
==========================================
- Coverage   71.62%   70.19%   -1.43%     
==========================================
  Files         149      149              
  Lines       32560    31919     -641     
==========================================
- Hits        23321    22406     -915     
- Misses       9239     9513     +274     

☔ 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.

3 participants