Skip to content
This repository has been archived by the owner on Oct 1, 2024. It is now read-only.

Commit

Permalink
revert more changes
Browse files Browse the repository at this point in the history
  • Loading branch information
Schaeff committed Sep 12, 2024
1 parent 0e13d88 commit 16d55a3
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 5 deletions.
12 changes: 8 additions & 4 deletions dft/benches/fft.rs
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ where
Dft: TwoAdicSubgroupDft<F>,
Standard: Distribution<F>,
{
let mut group = c.benchmark_group(format!(
let mut group = c.benchmark_group(&format!(
"fft::<{}, {}, {}>",
type_name::<F>(),
type_name::<Dft>(),
Expand Down Expand Up @@ -75,7 +75,11 @@ where
Dft: TwoAdicSubgroupDft<Complex<Mersenne31>>,
Standard: Distribution<Mersenne31>,
{
let mut group = c.benchmark_group(format!("m31_fft::<{}, {}>", type_name::<Dft>(), BATCH_SIZE));
let mut group = c.benchmark_group(&format!(
"m31_fft::<{}, {}>",
type_name::<Dft>(),
BATCH_SIZE
));
group.sample_size(10);

let mut rng = thread_rng();
Expand All @@ -98,7 +102,7 @@ where
Dft: TwoAdicSubgroupDft<F>,
Standard: Distribution<F>,
{
let mut group = c.benchmark_group(format!(
let mut group = c.benchmark_group(&format!(
"ifft::<{}, {}, {}>",
type_name::<F>(),
type_name::<Dft>(),
Expand Down Expand Up @@ -127,7 +131,7 @@ where
Dft: TwoAdicSubgroupDft<F>,
Standard: Distribution<F>,
{
let mut group = c.benchmark_group(format!(
let mut group = c.benchmark_group(&format!(
"coset_lde::<{}, {}, {}>",
type_name::<F>(),
type_name::<Dft>(),
Expand Down
1 change: 0 additions & 1 deletion monty-31/src/poseidon2.rs
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ use p3_symmetric::Permutation;
use crate::{monty_reduce, FieldParameters, MontyField31, MontyParameters};

/// Everything needed to compute multiplication by a WIDTH x WIDTH diffusion matrix whose monty form is 1 + Diag(vec).
///
/// vec is assumed to be of the form [-2, ...] with all entries after the first being small powers of 2.
pub trait DiffusionMatrixParameters<FP: FieldParameters, const WIDTH: usize>: Clone + Sync {
// Most of the time, ArrayLike will be [u8; WIDTH - 1].
Expand Down

0 comments on commit 16d55a3

Please sign in to comment.