I noticed that the next_power_of_two call in the following code is redundant:
let N = (0..sparse_polys.len())
.map(|i| sparse_polys[i].get_num_nz_entries())
.max()
.unwrap()
.next_power_of_two();
Since next_power_of_two is already being called in the get_num_nz_entries method, it can be safely removed from this code without affecting performance. Therefore, I have made a pull request to remove this redundant call.
I noticed that the
next_power_of_twocall in the following code is redundant:Since
next_power_of_twois already being called in theget_num_nz_entriesmethod, it can be safely removed from this code without affecting performance. Therefore, I have made a pull request to remove this redundant call.Thank you!