-
Notifications
You must be signed in to change notification settings - Fork 35
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
Mantainance #202
Mantainance #202
Conversation
let mut max_balance = max_value.clone(); | ||
|
||
// Perform the summation for each level up the Merkle Root | ||
for _ in 0..n_levels { |
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.
Can be replaced by a product
zk_prover/src/chips/range/tests.rs
Outdated
@@ -103,6 +103,32 @@ struct TestCircuit<const N_BYTES: usize> { | |||
pub b: Fp, | |||
} | |||
|
|||
impl<const N_BYTES: usize> TestCircuit<N_BYTES> { |
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.
My fault, haven't noticed this while reviewing the previous PR. Since there's actually some common code shared between different circuits, it can stay in CircuitBase 😄
c2a3ee2
into
v1-improvements-and-consolidation
Closes:
LEVELS
andN_BYTES
possible overflow combination insidegen_verifier
#201N_ASSETS
naming #200signatures.csv
out ofapi
folder inbackend
#196As consequence of that
csv
files folder for Benchmark #197N_ASSETS
fix naming summa-aggregation#9