Dev Ojha

Perhaps use https://cr.yp.to/papers/pippenger.pdf for an even faster exponentiation algorithm. See zcash issue 3128 for a potential discussion to see if this can be done in a side channel resistant manner....

The various authentication schemes have been explained in https://github.com/Project-Arda/bgls/commit/0e87e8820c6d0daec9354a440d8cdc8477cefffc. The API explanation has some explanation from the godocs also. (Not sure if more may or may not be needed)

I added an explanation of multi signatures vs aggregate signatures in https://github.com/Project-Arda/bgls/commit/c61bcd60cbfce6bc466ee41804e1a4f7e1c38f8b . Also, the readme needs to be split up between bgls, and the curve implementation details, and placed...

I think `conditional_enforce_equal(&self, should_enforce, other)` is good, since the code looks like `a.conditional_enforce_equal(condition, b)`. And then code also looks consistent if you have a `Type::conditionally_select` nearby as well. Agreed on...

Oh that PR didn't actually change the FFT algorithm, just documented the existing one and identified the problem I'll let you know once I update the algorithm

I agree that this suggestion should be implemented.

Making Tendermint support aggregated signatures at the consensus level & default light client proof generation is also an option, that Tendermint team would probably be incredibly supportive of

> My reading on this so far is that the epoch block has a lot of computation (distributing the lp rewards) and results in the emission of a lot of...

To improve the time.Now() call, we can make that a telemetry function. So replace `time.Now()` in the defer statements with `telemetry.StartTimer()`. And the StartTimer function would be `if telemetry enabled...

Does anyone have any guidance on how we can get there to be a flag for whether or not telemetry is enabled? Some thoughts that come to mind for me:...