-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
DNM: Decimal error measurements and benchmarks #7772
Conversation
Codecov Report
@@ Coverage Diff @@
## master #7772 +/- ##
==========================================
+ Coverage 54.31% 61.57% +7.26%
==========================================
Files 610 601 -9
Lines 38621 34445 -4176
==========================================
+ Hits 20976 21210 +234
+ Misses 15508 11052 -4456
- Partials 2137 2183 +46
|
TLDR:
|
I don't like our decimal implementation one bit. There is an issue floating around somewhere where we propose to perform arithmetic in arbitrary precision yielding a final result in fixed precision. Would be nice to tackle that and use a more robust decimal implementation. |
Some more benchmarks on specific operations (+, -, *, /). Basically this generates some random floating point numbers and performs the operation ( apd's
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
ref #7640
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/
) or specification (x/<module>/spec/
)godoc
comments.Unreleased
section inCHANGELOG.md
Files changed
in the Github PR explorerCodecov Report
in the comment section below once CI passes