-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add MPFR trans functions #249
Conversation
ab4080a
to
a8d46d0
Compare
I actually want to get #250 in before this one. Essentially: I am still tweaking gas a bit, and that PR allows for gassing |
0c88323
to
65d101a
Compare
65d101a
to
95e5b99
Compare
let result = Musl.trans_logBase (dec2F base) (dec2F arg) | ||
guardNanOrInf info result | ||
returnCEKValue cont handler (VLiteral (LDecimal (f2Dec result))) | ||
result <- guardNanOrInf info $ MPFR.mpfr_log base arg |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
args -> argsError info b args | ||
where | ||
checkArgAndCompute n = do | ||
when (n <= 0) $ throwExecutionError info (ArithmeticException "Natural logarithm must be greater then 0") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
7347f0f
to
11fa505
Compare
Based on https://github.com/kadena-io/pact/pull/1045/files#diff-9619507e39a18e5493fafa72986db085aebadb61dc76aac3396fc34e30d38623
PR checklist:
Additionally, please justify why you should or should not do the following: