-
-
Notifications
You must be signed in to change notification settings - Fork 47
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
Productionize KZG EIP-4844 #304
Conversation
@mratsim let me know if you have any questions regarding adding |
3-way benchmarks vs c-kzg-4844 (cc @jtraglia) and go-kzg-4844 (cc @kevaundray)
|
Impressive numbers! Howevere, |
Not fuzzed yet but I do pass the tests, I assume it's my random values that are weird because the code is really just forwarding to the MSM. |
To my eyes, it appears that blobs are only initialized with a single field element. This would explain things. constantine/benchmarks/bench_ethereum_eip4844_kzg.nim Lines 46 to 52 in c784928
|
Good catch, this was it, new bench (didn't rebench c-kzg-4844 and go-kzg-4844)
|
Nice, that's still really impressive. Good job! |
Turned out I wasn't parallelizing kzg batch verification but only preprocessing. See a1ced2d There is a whopping 52% speedup on parallel batch verification.
|
PR too long due to trusted setup updates and .yaml test vectors. The parallel C API, Go and Rust bindings will be provided in a separate. |
This addresses #300