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

Circle protocol #938

Draft
wants to merge 107 commits into
base: main
Choose a base branch
from
Draft

Circle protocol #938

wants to merge 107 commits into from

Conversation

ColoCarletti
Copy link
Contributor

TITLE

Description

Description of the pull request changes and motivation.

Type of change

Please delete options that are not relevant.

  • New feature
  • Bug fix
  • Optimization

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

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 99.14530% with 1 line in your changes missing coverage. Please review.

Project coverage is 71.74%. Comparing base (a6738c7) to head (3b165ee).

Files with missing lines Patch % Lines
math/src/circle/polynomial.rs 97.56% 1 Missing ⚠️
Additional details and impacted files
@@                Coverage Diff                @@
##           circle-starks     #938      +/-   ##
=================================================
+ Coverage          71.62%   71.74%   +0.11%     
=================================================
  Files                149      151       +2     
  Lines              32560    32677     +117     
=================================================
+ Hits               23321    23443     +122     
+ Misses              9239     9234       -5     

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

Base automatically changed from circle-starks to main November 4, 2024 13:26
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.

4 participants