Skip to content

Commit

Permalink
fix(sc): synthesize: CS->Config+Region
Browse files Browse the repository at this point in the history
The old design didn't take into account that we don't
need a constraint system, but rather that the table
region is what we need
  • Loading branch information
cyphersnake committed Oct 5, 2023
1 parent 653ef8c commit dc7f373
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions src/step_circuit.rs
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
use ff::PrimeField;
use halo2_proofs::{
circuit::AssignedCell,
circuit::{AssignedCell, Region},
plonk::{Circuit, ConstraintSystem},
};

Expand Down Expand Up @@ -42,7 +42,7 @@ pub trait StepCircuit<const ARITY: usize, F: PrimeField>: Circuit<F> {
/// columns.
///
/// This setup is crucial for the functioning of the IVC-based system.
fn configure(cs: &mut ConstraintSystem<F>) -> <Self as StepCircuit<ARITY, F>>::StepConfig;
fn configure(cs: &mut ConstraintSystem<F>) -> Self::StepConfig;

/// Sythesize the circuit for a computation step and return variable
/// that corresponds to the output of the step z_{i+1}
Expand All @@ -51,7 +51,8 @@ pub trait StepCircuit<const ARITY: usize, F: PrimeField>: Circuit<F> {
/// Return `z_out` result
fn synthesize(
&self,
cs: &mut ConstraintSystem<F>,
config: Self::Config,
region: Region<'_, F>,
z_in: &[AssignedCell<F, F>; ARITY],
) -> Result<[AssignedCell<F, F>; ARITY], SynthesisError>;

Expand Down

0 comments on commit dc7f373

Please sign in to comment.