Closed p0mvn closed 1 year ago
Closes: #XXX
Progress towards: https://github.com/osmosis-labs/osmosis/issues/4645
There is currently no MulRoundUp function defined on the decimal.
MulRoundUp
It is useful to have this API for https://github.com/osmosis-labs/osmosis/issues/4645 as it will avoid us having to rely on Ceil() instead, losing accuracy and refactoring many tests.
Ceil()
MulRoundUpMut
This change added tests and can be verified as follows:
Unreleased
CHANGELOG.md
Closes: #XXX
What is the purpose of the change
Progress towards: https://github.com/osmosis-labs/osmosis/issues/4645
There is currently no
MulRoundUp
function defined on the decimal.It is useful to have this API for https://github.com/osmosis-labs/osmosis/issues/4645 as it will avoid us having to rely on
Ceil()
instead, losing accuracy and refactoring many tests.Brief Changelog
MulRoundUp
andMulRoundUpMut
Testing and Verifying
This change added tests and can be verified as follows:
Documentation and Release Note
Unreleased
section inCHANGELOG.md
? no