Henry de Valence

> All right, so if I understand correctly, you're advocating for getting rid of `tendermint::Time` in lieu of `tendermint_proto::Timestamp` since there's no real added value. No, sorry for being unclear,...

> If you want to use it directly, there's no way to add additional functionality to it I don't think this is quite true, it's possible to monkey-patch additional functionality...

> A case can be made for Time becoming more of an operational data type, optimized for arithmetic and comparisons which are performed on this type, for example, in the...

Thanks for the bump, sorry I missed this in December! On the argument ordering front, I think we could just swap the argument ordering to be consistent as you suggest,...

> We will need to examine the changes in this PR further and likely consider adding a Poseidon duplex wrapper that is less "raw" than what arkworks-rs currently has. I...

(A rendered copy of the docs is available here: https://rustdoc.penumbra.zone/main/ark_sponge/poseidon/index.html)

> Also, is there something specific that you'd like to get feedback on? Yeah, the main thing is whether this general approach seems good, before doing the work of also...

Hey, just bumping this -- if this approach seems good I can also apply it to the constraint implementations.

This is an interesting proposal. I wanted to flag, however, that it doesn't serve Penumbra's needs, and while I can't speak on behalf of other users I am unconvinced that...