-
Notifications
You must be signed in to change notification settings - Fork 77
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(tlsn): mpz upgrade #512
Conversation
3ef0b96
to
a7d45fd
Compare
@sinui0 , I think the conflicts need to be resolved. i cant load up this branch into vscode currently. |
a7d45fd
to
d9bc9e8
Compare
@themighty1 can you elaborate? Merge conflicts don't prevent you from checking out the branch |
d9bc9e8
to
80b258d
Compare
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.
nice work!!! i left some non-blocking qs
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.
Very nice 🚀 🚀
* refactor(tlsn): mpz upgrade * PR feedback
* refactor(tlsn): mpz upgrade * PR feedback
This PR updates
tlsn
crates to integrate thempz
upgrade.note: cherry pick #499 to run integration tests
Changes
serio
and newuid-mux
, clean up IO code including better error handling. This should also remove any potential race conditions that may occur from the remote peer closing.tracing
is always onforce-st
feature totlsn-prover
andtlsn-verifier
which forces single threaded execution (disables rayon)tlsn-common
to reduce noise