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
port local-{min,max}ima, estimate-global-{min, max} from scmutils [in progress] #108
Conversation
05b1b78
to
6524130
Compare
0024c6f
to
9dbaa83
Compare
6524130
to
f3e4d7b
Compare
9dbaa83
to
e733a9c
Compare
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.
These are all so appealing I want to start using them.
src/sicmutils/numbers.cljc
Outdated
(op (u/bigint a) b)) | ||
|
||
;; For NON integrals, we currently have no choice but to downcast the | ||
;; BigInt to a floating point number. TODO if we introduce BigDecimal |
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.
I guess it would depend on who you want to win. In the BigDecimal case, you probably get a horrible fraction out of the float, and the horrible fraction goes on to color everything it touches like a dye marker. At least having float be the absorbing state is closer to the hardware. As an occasional Mathematica user I'm always amused when something I type comes back with Brobdingnagian fractions in it, because I forgot to put a .
after one of the numbers to calm it down (or wrap with N[]
, but then you've paid the cost for a huge amount of ultimately uninteresting rational arithmetic.
e733a9c
to
a20e7a2
Compare
a20e7a2
to
96ec24f
Compare
Codecov Report
@@ Coverage Diff @@
## master #108 +/- ##
=======================================
Coverage 84.18% 84.18%
=======================================
Files 81 81
Lines 8434 8434
Branches 453 453
=======================================
Hits 7100 7100
Misses 881 881
Partials 453 453 Continue to review full report at Codecov.
|
afa6dd7
to
c33ef42
Compare
Moved to Emmy. |
This will round out the functions in
unimin.scm
, so I can't help but port them over.