-
Notifications
You must be signed in to change notification settings - Fork 145
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
Tracking PR for v0.2 release #8
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
docs(mdbook): update co-processor diagrams to show internal selectors
- turn on keccak256_bit_interleaving & keccak256_2_to_1_hash tests
refactor(assembly): speed up script compilation
cargo fix change string to str fix comment fixes change analyze function return type to result remove std remove std add comments
Add program analysis tool
Add op batch flags to decoder trace
docs(mdbook): fix rendering issue of pow2 aux table
Implement construction of decoder aux column `p3`
remove redundant function remove redundant function fixes docs(air): set first row address to 1 small fix small fix doc fixes
Add decoder constraint descriptions to mdbook
feat(air): add boundary constraint to hasher
Signed-off-by: Anjan Roy <hello@itzmeanjan.in>
Signed-off-by: Anjan Roy <hello@itzmeanjan.in>
Refactor hash chiplet constants and chiplets trace tests module
Update design section in mdbook
Add hasher lookups to the chiplets bus
`secp256k1` Curve Arithmetics
The auxiliary column `b_range` must always be one row longer than the main trace, in order to correctly verify range checks of the final value (u16::MAX). This is because the data used to generate the value included in the `b_range` column comes from the "current" row, but the value itself is included into the "next" row. This was resolved by padding the main range checker trace with an extra row of the final value (u16::MAX). The 8-bit segment of the Range Checker's trace was adjusted accordingly with an additional lookup of ZERO (to account for the transition between the final two rows of the 16-bit segment of the trace where the value changes by 0).
Fix range checker auxiliary column trace
Use u32assert2 in u64 stdlib
Update readme files
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR will track progress of v0.2 release of Miden VM.
p0
in the range checker #230p1
in the range checker #234p3
#260p1
andp2
#267p1
#280q
andp1
#286p0
in the range checker #230p1
in the range checker #234