Skip Menu |
 

This queue is for tickets about the Math-BigInt CPAN distribution.

Report information
The Basics
Id: 130720
Status: open
Priority: 0/
Queue: Math-BigInt

People
Owner: Nobody in particular
Requestors: ANDK [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 1.999817
Fixed in: (no value)



Subject: 1.999817 breaks the tests of CryptX, Math-BigInt-Lite, and Math-Counting
MIME-Version: 1.0
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
Message-ID: <rt-4.0.18-24819-1571274551-1974.0-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 408
Download (untitled) / with headers
text/plain 408b
As per subject. Involved distros: MIK/CryptX-0.064.tar.gz GJACKLAM/Math-BigInt-Lite-0.18.tar.gz MENE/Math-Counting-0.1305.tar.gz A sample fail report for CryptX: http://www.cpantesters.org/cpan/report/d7f2f25c-ed85-11e9-84a2-b83f75811fd8 (at the moment not reachable). Sample fail reports for Math-BigInt-Lite and Math-Counting are currently not publicly available, since cpantesters has a major outage.
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-24819-1571274551-1974.0-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <rt-4.0.18-24819-1571274551-1974.0-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-21241-1571322487-1789.130720-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 147
Download (untitled) / with headers
text/plain 147b
Thanks for notifying me! I tested Math-BigInt-1.999817 against Math-BigInt-Lite, so that is a fail that suprises me. Anyway, I will look into this.
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-21241-1571322487-1789.130720-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <rt-4.0.18-24819-1571274551-1974.0-0-0 [...] rt.cpan.org> <rt-4.0.18-21241-1571322487-1789.130720-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-13274-1571388616-1921.130720-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 755
Download (untitled) / with headers
text/plain 755b
All failed tests in CryptX, Math-BigInt-Lite, and Math-Counting are due to a change in the bpow() method (and the overloaded ** operator) regarding non-integer results. When a Math::BigInt operation results in a non-integer, the output is either truncated (i.e., rounded towards zero) or upgraded, if upgrading is enabled. The bpow() method did not adhere to this and returned NaN for things like 2**(-3). Compare this to explicitly computing 1/8, which returned 0. In 1.999817 this inconsistency was removed. So now, 2**(-3) and 1/(2**3) give the same result, i.e., 0. I consider this change in Math-BigInt to be a bug fix. I suggest the test suits for CryptX, Math-BigInt-Lite, and Math-Counting are updated to reflect this improvement of Math-BigInt.
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-24819-1571274551-1974.0-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <rt-4.0.18-24819-1571274551-1974.0-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-16545-1571453575-1619.130720-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 125
Download (untitled) / with headers
text/plain 125b
Could you find out why your own testing with Math-BigInt-Lite did not fail? Maybe there's something hidden below the surface?
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-16545-1571453575-1619.130720-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <rt-4.0.18-24819-1571274551-1974.0-0-0 [...] rt.cpan.org> <rt-4.0.18-16545-1571453575-1619.130720-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-17857-1571466287-660.130720-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 395
Download (untitled) / with headers
text/plain 395b
On Fri Oct 18 22:52:55 2019, ANDK wrote: Show quoted text
> Could you find out why your own testing with Math-BigInt-Lite did not > fail? Maybe there's something hidden below the surface?
I was unfortunately testing against my private Math-BigInt-Lite repo, which had been modified since the 0.18 release to reflect the changes to bpow()/**. When I test against the 0.18 release I see all the same failed tests.
MIME-Version: 1.0
In-Reply-To: <rt-4.0.18-24819-1571274551-1974.0-0-0 [...] rt.cpan.org>
X-Mailer: MIME-tools 5.504 (Entity 5.504)
Content-Disposition: inline
X-RT-Interface: Web
References: <rt-4.0.18-24819-1571274551-1974.0-0-0 [...] rt.cpan.org>
Content-Type: text/plain; charset="utf-8"
Message-ID: <rt-4.0.18-5690-1571845247-651.130720-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
X-RT-Encrypt: 0
X-RT-Sign: 0
Content-Length: 67
Thanks for the heads up. Fixed in Math-Counting-0.1306 now. - GENE


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.