-
Notifications
You must be signed in to change notification settings - Fork 144
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 STARK for small fields #945
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #945 +/- ##
==========================================
+ Coverage 71.12% 71.14% +0.02%
==========================================
Files 153 153
Lines 33101 33129 +28
==========================================
+ Hits 23542 23570 +28
Misses 9559 9559 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! really liked the TransitionEvaluationContext
solution
Refactor STARK for small fields
This PR refactors the
Air
andTransitionConstraints
traits to make the STARKs protocol compatible with small fields and their extensions.Type of change
Please delete options that are not relevant.