You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There are some real time overheads in Concentrated liquidity, involving converting Dec's to BigDec and BigDec's to Dec's. We do many more heap allocations than necessary.
These can almost always be made mutative. This can in principle offer a .5% sync speed speedup, but this will go down with the hopeful protorev restructuring.
Background
There are some real time overheads in Concentrated liquidity, involving converting Dec's to BigDec and BigDec's to Dec's. We do many more heap allocations than necessary.
These can almost always be made mutative. This can in principle offer a .5% sync speed speedup, but this will go down with the hopeful protorev restructuring.
To make this work well, we need to create BigIntMut() API's on LegacyDec upstream, similar to what is done for Int. See here: https://github.com/cosmos/cosmos-sdk/blob/main/math/int.go#L105-L111 . Once that exists, we can do this well.
Acceptance Criteria
Faster mutative BigDec.DecMut() and BigDec.FromDecMut() functions.
The text was updated successfully, but these errors were encountered: