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
bignum fails to compute log(2) #14908
Comments
From ntysdd@gmail.comPerl version : 5.22.0 / MSWin32-x64-multi-thread
|
From @jkeenanOn Tue Sep 15 11:08:49 2015, ntysdd@gmail.com wrote:
And what's even more unfortunate is the fact that this is cited in the documentation as something "to impress the Python crowd." ;-( From ./dist/bignum/lib/bignum.pm in blead: #####
-- |
The RT System itself - Status changed from 'new' to 'open' |
From @pjacklamI am aware of this bug, and I already have a patch and a test case. This bug was introduced when I fixed a different bug, and there was no test Here is the underlying problem: The pragmas big(num|int|rat) are supposed As of right now I see two possibilities: I haven't decided on which is the best. Input - of course including other Peter 2015-09-16 2:44 GMT+02:00 James E Keenan via RT <perlbug-followup@perl.org>:
|
From perl@profvince.comLe 16/09/2015 10:03, Peter John Acklam a écrit :
Apologies if I'm oversimplifying this, but it just looks like the 'log' Vincent |
From @pjacklamI'm afraid that is not enough. log() calls blog(), which in turn calls One option is to modify objectify() so undefs pass through unmodified, but I also realize now that the Math-BigInt distribution contains NO tests for |
From @ap* Peter John Acklam <pjacklam@gmail.com> [2015-09-17 14:30]:
This is also a breaking change to the API: any downstream users of the And there is no benefit in “punishing” such “wrongdoers” – esp. if the Sometimes you have to break a few eggs to make an omelette, but that C.f. http://neilb.org/tag/cpan-river/
I am opining without any real knowledge of the codebase so I don’t know Regards, |
From @pjacklamPlease try Math-BigInt-1.999702. The new release also fixes the case log(inf), which now returns inf, not Peter |
Migrated from rt.perl.org#126061 (status was 'open')
Searchable as RT126061$
The text was updated successfully, but these errors were encountered: