-
Notifications
You must be signed in to change notification settings - Fork 174
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
Range checker improvements #395
Comments
I think I figured out how we can make this work without having to increase the minimum trace length to 16K cycles (and maybe we can even decrease it to be smaller than 1K cycles). The core of the idea in the original post is that we allow But what if instead we allow This is still a degree |
Ah, makes sense. |
Closed by #949 |
In the current implementation, the range checker component (described here) consists of two tables: 8-bit table and 16-bit table. We use a selector column to identify which table we are in.
This structure can be greatly simplified if we get rid of the 8-bit table entirely, and instead, use a single constraint to allow "jumps" between two consecutive rows of the 16-bit table to be at most 8. Denoting$\Delta v = v' - v$ , the constraint would look as follows:
This constraint would have degree 9, which is fine for us.
The advantages of this approach:
The drawbacks of this approach:
The text was updated successfully, but these errors were encountered: