Subject: Re: [boost] [Review Request] Multiprecision Arithmetic Library
From: Christopher Kormanyos (e_float_at_[hidden])
Date: 2012-04-12 16:19:44
>> Hmm, that fails a number of tests with error rates > 10^10 - see for
>> example test_exp.cpp. Can you investigate?
> Never mind - I think I have it - needed to replace uses of cpp_dec_float_max_digits10
> with cpp_dec_float_total_digits10 which does what it says,
> where as cpp_dec_float_max_digits10 doesn't anymore.
> With that change the tests I've run pass so far.... but they're still running.
> Maybe we should rename cpp_dec_float_max_digits10 to something else?
Or eliminate it because it is redundant.
I will report soon.
> BTW, what's the logic behind the +1 in defining that one?
> Not that it makes much difference either which way.
It's redundant and not needed, as addition of +1
should be done in std::numeric_limits anyway.
> Cheers, John.
Thanks again, John.
I got the tests running. What a nice test suite!
I need to run the tests myself and know if the tests are OK
before suggesting any changes.
Is there a collected summary of the output files?
Is this target report at the end of the test run of concern?
...failed updating 7 targets...
...skipped 21 targets...
...updated 574 targets...
Best regards, Chris.
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk