fix: removed leftover; proper variable name in benchmark

summa-dev · Mar 15, 2024 · 6289153 · 6289153
1 parent c831f5d
commit 6289153
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/kzg_prover/benches/kzg.rs b/kzg_prover/benches/kzg.rs
@@ -56,11 +56,11 @@ fn bench_kzg<
     let entries = generate_dummy_entries::<N_USERS, N_CURRENCIES>().unwrap();
 
     // Calculate total for all entry columns
-    let mut csv_total: Vec<BigUint> = vec![BigUint::from(0u32); N_CURRENCIES];
+    let mut total_balances: Vec<BigUint> = vec![BigUint::from(0u32); N_CURRENCIES];
 
     for entry in &entries {
         for (i, balance) in entry.balances().iter().enumerate() {
-            csv_total[i] += balance;
+            total_balances[i] += balance;
         }
     }
 
@@ -90,7 +90,7 @@ fn bench_kzg<
                     &advice_polys.advice_blinds,
                     &params,
                     balance_column_range,
-                    csv_total
+                    total_balances
                         .iter()
                         .map(|x| big_uint_to_fp(&(x)) * Fp::from(poly_length).invert().unwrap())
                         .collect::<Vec<Fp>>()
@@ -110,7 +110,7 @@ fn bench_kzg<
                     &advice_polys.advice_blinds,
                     &params,
                     balance_column_range,
-                    csv_total
+                    total_balances
                         .iter()
                         .map(|x| big_uint_to_fp(&(x)) * Fp::from(poly_length).invert().unwrap())
                         .collect::<Vec<Fp>>()
@@ -187,7 +187,7 @@ fn bench_kzg<
         &advice_polys.advice_blinds,
         &params,
         balance_column_range.clone(),
-        csv_total
+        total_balances
             .iter()
             .map(|x| big_uint_to_fp(&(x)) * Fp::from(poly_length).invert().unwrap())
             .collect::<Vec<Fp>>()

diff --git a/kzg_prover/src/utils/dummy_entries.rs b/kzg_prover/src/utils/dummy_entries.rs
@@ -6,8 +6,7 @@ use std::error::Error;
 use crate::entry::Entry;
 
 // This is for testing purposes with a large dataset instead of using a CSV file
-pub fn generate_dummy_entries<const N_USERS: usize, const N_CURRENCIES: usize>(// entries: &mut [Entry<N_CURRENCIES>],
-    // cryptocurrencies: &mut [Cryptocurrency],
+pub fn generate_dummy_entries<const N_USERS: usize, const N_CURRENCIES: usize>(
 ) -> Result<Vec<Entry<N_CURRENCIES>>, Box<dyn Error>> {
     // Ensure N_CURRENCIES is greater than 0.
     if N_CURRENCIES == 0 {