-
Notifications
You must be signed in to change notification settings - Fork 26
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 libsnark helpers [depends on #200] #206
Conversation
d11f25b
to
a79fd88
Compare
836ef4a
to
cfe6813
Compare
Removed the WIP. As above, this PR does not address refactoring of the serialization functions, or renaming. It aims to decide on the file divisions / locations and to expse the snarks in a slightly more versatile way. Further PRs will follow, but flag anything that you think makes sense to include here. |
cfe6813
to
c032915
Compare
c032915
to
71ba0ed
Compare
71ba0ed
to
e2dbc3b
Compare
All comments above should have been addressed now (commenting here rather than on all individual threads). |
LGTM |
Refactor libsnark helpers [depends on #200]
Rearranging files and the snark abstractions to parameterize by snark.
Part of a larger cleanup, this PR focussed on these two points and remaining clean up will happen in future PRs.