updates and addressed comments

This commit is contained in:
Nicholas Ward 2022-01-27 13:34:27 -08:00
parent edf7563208
commit 82e2872f5e
12 changed files with 24 additions and 302 deletions

View File

@ -259,3 +259,11 @@ impl<C: Curve> Neg for ProjectivePoint<C> {
ProjectivePoint { x, y: -y, z }
}
}
pub fn base_to_scalar<C: Curve>(x: C::BaseField) -> C::ScalarField {
C::ScalarField::from_biguint(x.to_biguint())
}
pub fn scalar_to_base<C: Curve>(x: C::ScalarField) -> C::BaseField {
C::BaseField::from_biguint(x.to_biguint())
}

View File

@ -1,7 +1,7 @@
use itertools::{unfold, Itertools};
use num::BigUint;
use crate::curve::curve_types::{AffinePoint, Curve, CurveScalar};
use crate::curve::curve_types::{base_to_scalar, AffinePoint, Curve, CurveScalar};
use crate::field::field_types::Field;
use crate::hash::hash_types::RichField;
use crate::hash::hashing::{hash_n_to_m, PlonkyPermutation};
@ -15,14 +15,6 @@ pub struct ECDSASignature<C: Curve> {
pub struct ECDSASecretKey<C: Curve>(pub C::ScalarField);
pub struct ECDSAPublicKey<C: Curve>(pub AffinePoint<C>);
pub fn base_to_scalar<C: Curve>(x: C::BaseField) -> C::ScalarField {
C::ScalarField::from_biguint(x.to_biguint())
}
pub fn scalar_to_base<C: Curve>(x: C::ScalarField) -> C::BaseField {
C::BaseField::from_biguint(x.to_biguint())
}
pub fn hash_to_bits<F: RichField, P: PlonkyPermutation<F>>(x: F, num_bits: usize) -> Vec<bool> {
let hashed = hash_n_to_m::<F, P>(&vec![x], 1, true)[0];

View File

@ -196,24 +196,6 @@ impl<F: RichField + Extendable<D>, const D: usize> CircuitBuilder<F, D> {
(output_result, output_borrow)
}
pub fn split_to_u32(&mut self, x: Target) -> (U32Target, U32Target) {
let low = self.add_virtual_u32_target();
let high = self.add_virtual_u32_target();
let base = self.constant(F::from_canonical_u64(1u64 << 32));
let combined = self.mul_add(high.0, base, low.0);
self.connect(x, combined);
self.add_simple_generator(SplitToU32Generator::<F, D> {
x: x.clone(),
low: low.clone(),
high: high.clone(),
_phantom: PhantomData,
});
(low, high)
}
}
#[derive(Debug)]
@ -235,7 +217,7 @@ impl<F: RichField + Extendable<D>, const D: usize> SimpleGenerator<F>
let x = witness.get_target(self.x.clone());
let x_u64 = x.to_canonical_u64();
let low = x_u64 as u32;
let high: u32 = (x_u64 >> 32).try_into().unwrap();
let high = (x_u64 >> 32) as u32;
out_buffer.set_u32_target(self.low.clone(), low);
out_buffer.set_u32_target(self.high.clone(), high);

View File

@ -1,6 +1,6 @@
use std::marker::PhantomData;
use num::{BigUint, FromPrimitive, Integer, Zero};
use num::{BigUint, Integer, Zero};
use plonky2_field::extension_field::Extendable;
use crate::gadgets::arithmetic_u32::U32Target;
@ -27,14 +27,7 @@ impl BigUintTarget {
impl<F: RichField + Extendable<D>, const D: usize> CircuitBuilder<F, D> {
pub fn constant_biguint(&mut self, value: &BigUint) -> BigUintTarget {
let base = BigUint::from_u64(1 << 32).unwrap();
let mut limb_values = Vec::new();
let mut current = value.clone();
while current > BigUint::zero() {
let (div, rem) = current.div_rem(&base);
current = div;
limb_values.push(rem.to_u64_digits()[0] as u32);
}
let limb_values = value.to_u32_digits();
let limbs = limb_values.iter().map(|&l| self.constant_u32(l)).collect();
BigUintTarget { limbs }
@ -167,11 +160,10 @@ impl<F: RichField + Extendable<D>, const D: usize> CircuitBuilder<F, D> {
let t = b.target;
BigUintTarget {
limbs: a
.limbs
.iter()
.map(|l| U32Target(self.mul(l.0, t)))
.collect(),
limbs: a.limbs
.iter()
.map(|&l| U32Target(self.mul(l.0, t)))
.collect(),
}
}

View File

@ -135,7 +135,7 @@ impl<F: RichField + Extendable<D>, const D: usize> CircuitBuilder<F, D> {
for &bit in bits.iter() {
let not_bit = self.not(bit);
let result_plus_2_i_p = self.curve_add(&result, &two_i_times_p);
let new_x_if_bit = self.mul_nonnative_by_bool(&result_plus_2_i_p.x, bit);
@ -321,6 +321,7 @@ mod tests {
builder.connect_affine_point(&neg_five_g_expected, &neg_five_g_actual);
println!("NUM GATES: {}", builder.num_gates());
let data = builder.build::<C>();
let proof = data.prove(pw).unwrap();

View File

@ -93,17 +93,8 @@ impl<F: RichField + Extendable<D>, const D: usize> CircuitBuilder<F, D> {
a: &NonNativeTarget<FF>,
b: BoolTarget,
) -> NonNativeTarget<FF> {
let t = b.target;
NonNativeTarget {
value: BigUintTarget {
limbs: a
.value
.limbs
.iter()
.map(|l| U32Target(self.mul(l.0, t)))
.collect(),
},
value: self.mul_biguint_by_bool(&a.value, b),
_phantom: PhantomData,
}
}

View File

@ -138,7 +138,7 @@ impl<F: RichField + Extendable<D>, const D: usize> Gate<F, D> for ArithmeticExte
);
g
})
.collect::<Vec<_>>()
.collect()
}
fn num_wires(&self) -> usize {

View File

@ -425,10 +425,7 @@ mod tests {
v1.append(&mut output_limbs_f);
}
v0.iter()
.chain(v1.iter())
.map(|&x| x.into())
.collect::<Vec<_>>()
v0.iter().chain(v1.iter()).map(|&x| x.into()).collect()
}
let mut rng = rand::thread_rng();

View File

@ -658,7 +658,7 @@ mod tests {
v.append(&mut intermediate_values);
v.append(&mut msd_bits);
v.iter().map(|&x| x.into()).collect::<Vec<_>>()
v.iter().map(|&x| x.into()).collect()
};
let mut rng = rand::thread_rng();

View File

@ -343,7 +343,7 @@ mod tests {
for i in 0..coeffs.len() {
v.extend(coeffs.coeffs[i].0);
}
v.iter().map(|&x| x.into()).collect::<Vec<_>>()
v.iter().map(|&x| x.into()).collect()
}
// Get a working row for InterpolationGate.

View File

@ -1,241 +0,0 @@
use std::marker::PhantomData;
use plonky2_field::extension_field::Extendable;
use crate::gates::gate::Gate;
use crate::gates::util::StridedConstraintConsumer;
use crate::hash::hash_types::RichField;
use crate::iop::ext_target::ExtensionTarget;
use crate::iop::generator::{GeneratedValues, SimpleGenerator, WitnessGenerator};
use crate::iop::target::Target;
use crate::iop::wire::Wire;
use crate::iop::witness::{PartitionWitness, Witness};
use crate::plonk::circuit_builder::CircuitBuilder;
use crate::plonk::vars::{EvaluationTargets, EvaluationVars, EvaluationVarsBase};
/// A gate to perform a basic mul-add on 32-bit values (we assume they are range-checked beforehand).
#[derive(Copy, Clone, Debug)]
pub struct MulBiguintBoolGate<F: RichField + Extendable<D>, const D: usize> {
pub num_limbs: usize,
_phantom: PhantomData<F>,
}
impl<F: RichField + Extendable<D>, const D: usize> MulBiguintBoolGate<F, D> {
pub fn new(num_limbs: usize) -> Self {
Self {
num_limbs,
_phantom: PhantomData,
}
}
pub fn wire_ith_input_limb(&self, i: usize) -> usize {
debug_assert!(i < self.num_limbs);
i
}
pub fn wire_input_bool(&self) -> usize {
self.num_limbs
}
pub fn wire_ith_output_limb(&self, i: usize) -> usize {
debug_assert!(i < self.num_limbs);
self.num_limbs + 1 + i
}
}
impl<F: RichField + Extendable<D>, const D: usize> Gate<F, D> for MulBiguintBoolGate<F, D> {
fn id(&self) -> String {
format!("{:?}", self)
}
fn eval_unfiltered(&self, vars: EvaluationVars<F, D>) -> Vec<F::Extension> {
let mut constraints = Vec::with_capacity(self.num_constraints());
let input_bool = vars.local_wires[self.wire_input_bool()];
for i in 0..self.num_limbs {
let input_i = vars.local_wires[self.wire_ith_input_limb(i)];
let output_i = vars.local_wires[self.wire_ith_output_limb(i)];
constraints.push(input_i * input_bool - output_i);
}
constraints
}
fn eval_unfiltered_base_one(
&self,
vars: EvaluationVarsBase<F>,
mut yield_constr: StridedConstraintConsumer<F>,
) {
let input_bool = vars.local_wires[self.wire_input_bool()];
for i in 0..self.num_limbs {
let input_i = vars.local_wires[self.wire_ith_input_limb(i)];
let output_i = vars.local_wires[self.wire_ith_output_limb(i)];
yield_constr.one(input_i * input_bool - output_i);
}
}
fn eval_unfiltered_recursively(
&self,
builder: &mut CircuitBuilder<F, D>,
vars: EvaluationTargets<D>,
) -> Vec<ExtensionTarget<D>> {
let mut constraints = Vec::with_capacity(self.num_constraints());
let input_bool = vars.local_wires[self.wire_input_bool()];
for i in 0..self.num_limbs {
let input_i = vars.local_wires[self.wire_ith_input_limb(i)];
let output_i = vars.local_wires[self.wire_ith_output_limb(i)];
constraints.push(builder.mul_sub_extension(input_i, input_bool, output_i));
}
constraints
}
fn generators(
&self,
gate_index: usize,
_local_constants: &[F],
) -> Vec<Box<dyn WitnessGenerator<F>>> {
let gen = MulBiguintBoolGenerator {
gate: *self,
gate_index,
_phantom: PhantomData,
};
vec![Box::new(gen.adapter())]
}
fn num_wires(&self) -> usize {
self.num_limbs * 2 + 1
}
fn num_constants(&self) -> usize {
0
}
fn degree(&self) -> usize {
2
}
fn num_constraints(&self) -> usize {
self.num_limbs
}
}
#[derive(Clone, Debug)]
struct MulBiguintBoolGenerator<F: RichField + Extendable<D>, const D: usize> {
gate: MulBiguintBoolGate<F, D>,
gate_index: usize,
_phantom: PhantomData<F>,
}
impl<F: RichField + Extendable<D>, const D: usize> SimpleGenerator<F>
for MulBiguintBoolGenerator<F, D>
{
fn dependencies(&self) -> Vec<Target> {
let local_target = |input| Target::wire(self.gate_index, input);
(0..self.gate.num_limbs)
.map(|i| local_target(self.gate.wire_ith_input_limb(i)))
.chain([local_target(self.gate.wire_input_bool())])
.collect()
}
fn run_once(&self, witness: &PartitionWitness<F>, out_buffer: &mut GeneratedValues<F>) {
let local_wire = |input| Wire {
gate: self.gate_index,
input,
};
let get_local_wire = |input| witness.get_wire(local_wire(input));
let input_bool = get_local_wire(self.gate.wire_input_bool());
for i in 0..self.gate.num_limbs {
let input_limb = get_local_wire(self.gate.wire_ith_input_limb(i));
let output_wire = local_wire(self.gate.wire_ith_output_limb(i));
let output_limb = input_limb * input_bool;
out_buffer.set_wire(output_wire, output_limb);
}
}
}
#[cfg(test)]
mod tests {
use std::marker::PhantomData;
use anyhow::Result;
use plonky2_field::field_types::Field;
use plonky2_field::goldilocks_field::GoldilocksField;
use rand::Rng;
use crate::gates::gate::Gate;
use crate::gates::gate_testing::{test_eval_fns, test_low_degree};
use crate::gates::mul_biguint_bool::MulBiguintBoolGate;
use crate::hash::hash_types::HashOut;
use crate::plonk::config::{GenericConfig, PoseidonGoldilocksConfig};
use crate::plonk::vars::EvaluationVars;
#[test]
fn low_degree() {
test_low_degree::<GoldilocksField, _, 4>(MulBiguintBoolGate::<GoldilocksField, 4> {
num_limbs: 8,
_phantom: PhantomData,
})
}
#[test]
fn eval_fns() -> Result<()> {
const D: usize = 2;
type C = PoseidonGoldilocksConfig;
type F = <C as GenericConfig<D>>::F;
test_eval_fns::<F, C, _, D>(MulBiguintBoolGate::<GoldilocksField, D> {
num_limbs: 8,
_phantom: PhantomData,
})
}
#[test]
fn test_gate_constraint() {
const D: usize = 2;
type C = PoseidonGoldilocksConfig;
type F = <C as GenericConfig<D>>::F;
type FF = <C as GenericConfig<D>>::FE;
const NUM_LIMBS: usize = 8;
fn get_wires(input_limbs: Vec<F>, input_bool: bool) -> Vec<FF> {
let output_limbs = input_limbs
.iter()
.map(|&l| if input_bool { l } else { F::ZERO });
input_limbs
.iter()
.cloned()
.chain([F::from_bool(input_bool)])
.chain(output_limbs)
.map(|x| x.into())
.collect()
}
let mut rng = rand::thread_rng();
let input_limbs: Vec<_> = (0..NUM_LIMBS)
.map(|_| F::from_canonical_u64(rng.gen()))
.collect();
let input_bool: bool = rng.gen();
let gate = MulBiguintBoolGate::<F, D> {
num_limbs: NUM_LIMBS,
_phantom: PhantomData,
};
let vars = EvaluationVars {
local_constants: &[],
local_wires: &get_wires(input_limbs, input_bool),
public_inputs_hash: &HashOut::rand(),
};
assert!(
gate.eval_unfiltered(vars).iter().all(|x| x.is_zero()),
"Gate constraints are not satisfied."
);
}
}

View File

@ -432,7 +432,7 @@ mod tests {
v.push(F::from_bool(switch));
}
v.iter().map(|&x| x.into()).collect::<Vec<_>>()
v.iter().map(|&x| x.into()).collect()
}
let first_inputs: Vec<Vec<F>> = (0..num_copies).map(|_| F::rand_vec(CHUNK_SIZE)).collect();