crate-crypto / go-eth-kzg

Apache License 2.0
29 stars 22 forks source link

Set each method to explicitly take the number of go-routines #46

Closed kevaundray closed 1 year ago

kevaundray commented 1 year ago

In reply to https://github.com/crate-crypto/go-kzg-4844/pull/45#issuecomment-1521413166

karalabe commented 1 year ago

Unfortunately this is not enough. Half of the concurrency is not even exposed in the underlying library and the config is ignored (i.e. https://github.com/ConsenSys/gnark-crypto/blob/master/ecc/bls12-381/multiexp.go#L164 has 4 goroutines spun up; https://github.com/ConsenSys/gnark-crypto/blob/master/ecc/bls12-381/multiexp.go#L421 also 4)