-
Notifications
You must be signed in to change notification settings - Fork 365
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 the code structure #155
Conversation
c4783ad
to
3bdf4e1
Compare
zkevm
crate to prover
zkevm
crate to prover
zkevm
crate to prover
zkevm
folder to prover
@@ -19,6 +19,7 @@ pub use outer_proof::AggCircuitProof; | |||
#[cfg(target_os = "linux")] | |||
extern crate procfs; | |||
|
|||
#[allow(dead_code)] |
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.
@lispc should we remove OPT_MEM
here given that it's not used anywhere?
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.
ok.. it was a feature when OPT_MEM enabled, release super circuit pk before start agg circuit proving. not needed now.
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.
Got it. So let's keep this line of code for now in case we may use it in the future?
This fix affects other pending PRs which use or update
zkevm
folder.Summary
According to #145 and discussing with @icemelon , suppose to rename
zkevm
crate (and folder) toprover
first (this PR), then work on this Asana task.Test
Clone new clean code and test with
make test-agg
(could pass) on super.