Ivo Kubjas

Results 165 comments of Ivo Kubjas

gnark now supports GPU acceleration, please see README and ICICLE docs.

@Tabaie, @gbotrel - ready for review.

@Tabaie - I added additionally false positive ignores for the statticcheck. Our linters have become more strict.

cc @Tabaie - a mental note to myself, that maybe I can reuse some utility functions from compression circuits for changing the endianness of the inputs.

Closing this PR. Linea prover calls the SHA2 permutation itself now and this wrapper is not needed.

> > so instead of hardcoding `SolverSucceeded` > > I think it is reasonable to always expect success. A single function is simpler than a whole circuit in that rather...

Thanks for all the changes! All good from my side now. I'll postpone the general interfaces for different curves for a separate PR (https://github.com/Consensys/gnark/issues/1303)

BLS glue itself is ready, but there are a few blockers: * to update to the latest zkevm_prague we need need go-corset dependency. * there is potentially memory regression with...

Sorry for the delay in reviewing the PR. I'm trying to do it soon 👍 .

To add, I think there definitely has to be something at root package `github.com/consensys/gnark`. Right now when looking at package doc then it is really confusing where to start from....