-
Notifications
You must be signed in to change notification settings - Fork 162
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
perf: sis tensor commitment #344
Conversation
…mitment Repush the modifs
Uses a `func() hash.Hash` instead of a `hash.Hash` when constructing the instance. Thus, this is a breaking change.
Add support for parallelization in the tensor-commitment
* fix NaiveMulMod so that the folder compiles * reference-benchmark * add theoritical measure * add a reference test for sparse polynomial
@ThomasPiellard on this branch it seems you added this:
for the fft. Also you mentioned an idea to store less things for the |
Yes for the twiddles you can only store the list |
without algorithmic change, just single threaded FFT and few less memallocs:
|
TODO:
|
see #252 #332 and #341 .
Notes
cc @ThomasPiellard
splitCoeffs
way of initializing the length of the result didn't work for other sis params.bn254
.Benchmarks
cc @AlexandreBelling