From 9ff39563483810444d8627f084d5b9f179bb9d86 Mon Sep 17 00:00:00 2001 From: Shahar Papini Date: Fri, 15 Mar 2024 07:25:47 +0200 Subject: [PATCH] DomainAccumulator allows accumulating a row at once --- src/core/air/accumulation.rs | 34 +++++++++++++++++++++++----------- src/fibonacci/component.rs | 7 ++++--- 2 files changed, 27 insertions(+), 14 deletions(-) diff --git a/src/core/air/accumulation.rs b/src/core/air/accumulation.rs index e89c6df11..776016032 100644 --- a/src/core/air/accumulation.rs +++ b/src/core/air/accumulation.rs @@ -4,7 +4,6 @@ //! f(p) = sum_i alpha^{N-1-i} u_i (P). use crate::core::backend::cpu::CPUCircleEvaluation; use crate::core::backend::{Backend, CPUBackend}; -use crate::core::fields::m31::BaseField; use crate::core::fields::qm31::SecureField; use crate::core::fields::secure::{SecureCirclePoly, SecureColumn}; use crate::core::fields::FieldExpOps; @@ -63,7 +62,7 @@ impl PointEvaluationAccumulator { /// Accumulates evaluations of u_i(P), each at an evaluation domain of the size of that polynomial. /// Computes the coefficients of f(P). pub struct DomainEvaluationAccumulator { - random_coeff: SecureField, + pub random_coeff: SecureField, /// Accumulated evaluations for each log_size. /// Each `sub_accumulation` holds `sum_{i=0}^{n-1} evaluation_i * alpha^(n-1-i)`, /// where `n` is the number of accumulated evaluations for this log_size. @@ -100,13 +99,18 @@ impl DomainEvaluationAccumulator { assert!(*log_size > 0 && *log_size < self.sub_accumulations.len() as u32); self.n_cols_per_size[*log_size as usize] += n_col; }); - self.sub_accumulations + let mut res = self + .sub_accumulations .get_many_mut(n_cols_per_size.map(|(log_size, _)| log_size as usize)) .unwrap_or_else(|e| panic!("invalid log_sizes: {}", e)) .map(|c| ColumnAccumulator { - random_coeff: self.random_coeff, + random_coeff_pow: self.random_coeff, col: c, - }) + }); + for i in 0..N { + res[i].random_coeff_pow = self.random_coeff.pow(n_cols_per_size[i].1 as u128); + } + res } /// Returns the log size of the resulting polynomial. @@ -154,12 +158,12 @@ impl DomainEvaluationAccumulator { /// An domain accumulator for polynomials of a single size. pub struct ColumnAccumulator<'a, B: Backend> { - random_coeff: SecureField, + random_coeff_pow: SecureField, col: &'a mut SecureColumn, } impl<'a> ColumnAccumulator<'a, CPUBackend> { - pub fn accumulate(&mut self, index: usize, evaluation: BaseField) { - let val = self.col.at(index) * self.random_coeff + evaluation; + pub fn accumulate(&mut self, index: usize, evaluation: SecureField) { + let val = self.col.at(index) * self.random_coeff_pow + evaluation; self.col.set(index, val); } } @@ -168,6 +172,7 @@ impl<'a> ColumnAccumulator<'a, CPUBackend> { mod tests { use std::array; + use num_traits::Zero; use rand::rngs::StdRng; use rand::{Rng, SeedableRng}; @@ -248,9 +253,16 @@ mod tests { (current_log_size, n_cols) }); let mut cols = accumulator.columns(n_cols_per_size); - for (log_size, evaluation) in log_sizes.iter().zip(evaluations.iter()) { - for (index, evaluation) in evaluation.iter().enumerate() { - cols[(log_size - LOG_SIZE_MIN) as usize].accumulate(index, *evaluation); + for log_size in n_cols_per_size.iter().map(|(log_size, _)| *log_size) { + for index in 0..(1 << log_size) { + let mut val = SecureField::zero(); + for (col_log_size, evaluation) in log_sizes.iter().zip(evaluations.iter()) { + if log_size != *col_log_size { + continue; + } + val = val * alpha + evaluation[index]; + } + cols[(log_size - LOG_SIZE_MIN) as usize].accumulate(index, val); } } let accumulator_poly = accumulator.finalize(); diff --git a/src/fibonacci/component.rs b/src/fibonacci/component.rs index 5f6b634cb..bf97083a9 100644 --- a/src/fibonacci/component.rs +++ b/src/fibonacci/component.rs @@ -83,6 +83,7 @@ impl Component for FibonacciComponent { trace: &ComponentTrace<'_, CPUBackend>, evaluation_accumulator: &mut DomainEvaluationAccumulator, ) { + let random_coeff = evaluation_accumulator.random_coeff; let poly = &trace.columns[0]; let trace_domain = CanonicCoset::new(self.log_size); let trace_eval_domain = CanonicCoset::new(self.log_size + 1).circle_domain(); @@ -90,7 +91,7 @@ impl Component for FibonacciComponent { // Step constraint. let constraint_log_degree_bound = trace_domain.log_size() + 1; - let [mut accum] = evaluation_accumulator.columns([(constraint_log_degree_bound, 1)]); + let [mut accum] = evaluation_accumulator.columns([(constraint_log_degree_bound, 2)]); let constraint_eval_domain = trace_eval_domain; for (off, point_coset) in [ (0, constraint_eval_domain.half_coset), @@ -104,8 +105,8 @@ impl Component for FibonacciComponent { for (i, point) in point_coset.iter().enumerate() { let mask = [eval[i], eval[i as isize + mul], eval[i as isize + 2 * mul]]; let res = self.step_constraint_eval_quotient_by_mask(point, &mask); - accum.accumulate(bit_reverse_index(i + off, constraint_log_degree_bound), res); - let res = self.boundary_constraint_eval_quotient_by_mask(point, &[mask[0]]); + let res = res * random_coeff + + self.boundary_constraint_eval_quotient_by_mask(point, &[mask[0]]); accum.accumulate(bit_reverse_index(i + off, constraint_log_degree_bound), res); } }