Small change that was left out of construct_addition_chains fix (Azte…

…cProtocol/barretenberg#404) Fixes the intermittent construct_addition_chains bugs (the previous fix was incomplete) and cleans the test up a bit
AztecProtocol · May 3, 2023 · 166863d · 166863d
1 parent 5011d9b
commit 166863d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/...pp/src/barretenberg/ecc/curves/bn254/scalar_multiplication/scalar_multiplication.test.cpp b/...pp/src/barretenberg/ecc/curves/bn254/scalar_multiplication/scalar_multiplication.test.cpp
@@ -482,15 +482,15 @@ TEST(scalar_multiplication, construct_addition_chains)
                                                                        bucket_counts,
                                                                        &bit_offsets[0],
                                                                        state.point_schedule,
-                                                                       num_points,
+                                                                       static_cast<uint32_t>(state.round_counts[0]),
                                                                        static_cast<uint32_t>(num_buckets),
                                                                        bucket_empty_status };
 
     start = std::chrono::steady_clock::now();
     scalar_multiplication::construct_addition_chains(product_state, true);
-    // scalar_multiplication::scalar_multiplication_internal<num_points>(state, monomials);
     end = std::chrono::steady_clock::now();
     diff = std::chrono::duration_cast<std::chrono::milliseconds>(end - start);
+    info("construct addition chains: ", diff.count(), "ms");
     std::cout << "scalar mul: " << diff.count() << "ms" << std::endl;
 
     aligned_free(bucket_empty_status);