Ed: I am fine TODO-ing those failing tests. I see you've already done that.
Thanks.
Chris: My hunch is that it doesn't have anything to do with 64-bit index
support, rather just what datatypes can hold how big of values on what kind of
Perls. My first idea was to TODO those tests for Perls where ivsize<8, but
according to Ed's 64-bit Linux vs Strawberry comparison that won't work. Needs
some more thinks...
Perl has similar tests at
http://perl5.git.perl.org/perl.git/blob/HEAD:/t/opbasic/arith.t
<http://perl5.git.perl.org/perl.git/blob/HEAD:/t/opbasic/arith.t>, lines 68-77.
Derek
> On Mar 15, 2015, at 1:39 PM, Ed <ej...@hotmail.com> wrote:
>
> Yes, I’ve been able to confirm that:
> https://github.com/PDLPorters/pdl/pull/75
> <https://github.com/PDLPorters/pdl/pull/75>
>
> Thanks! I’ll be TODO-ing that and once I’ve got a review and green light from
> Travis (and my local test on Win32) I’ll be merging.
>
> From: Chris Marshall <mailto:devel.chm...@gmail.com>
> Sent: Sunday, March 15, 2015 5:55 PM
> To: Ed <mailto:ej...@hotmail.com> ; pdl-devel
> <mailto:pdl-devel@lists.sourceforge.net>
> Subject: Re: [Pdl-devel] t/ops.t fail on 64-bit Strawberry Perl
>
> I think Rob determined that these failures are the same as for PDL-2.007.
> I recommend marking them TODO and address with PDL-2.009. I think
> that the source of the error is in the remaining 64bit index support work...
>
> --Chris
>
> On 3/15/2015 12:43, Ed wrote:
>> Dear PDL devs,
>>
>> This snippet of t/ops.t (I changed it to cmp_ok so I can see what the actual
>> value is, see branch intmaxfix):
>>
>> $INT_MAX=2147483647;
>> cmp_ok indx($INT_MAX*4)%2, '==', 0, 'big indx modulus';
>> cmp_ok double($INT_MAX*4)%2, '==', 0, 'big double modulus';
>>
>> It passes on my 64-bit Linux Perl 5.20.1, but on my 64-bit Strawberry Perl
>> 5.20.0 gives these results:
>> # Failed test 'big indx modulus'
>> # at t/ops.t line 165.
>> # got: 8589934592
>> # expected: 0
>>
>> # Failed test 'big double modulus'
>> # at t/ops.t line 168.
>> # got: 12884901884
>> # expected: 0
>>
>> Derek, you last changed t/ops.t, do you have any insight on this? Is this a
>> test problem, or an underlying code problem? Please feel free to push to my
>> branch intmaxfix if you have code.
>>
>> Best regards,
>> Ed
>>
>>
>> ------------------------------------------------------------------------------
>> Dive into the World of Parallel Programming The Go Parallel Website,
>> sponsored
>> by Intel and developed in partnership with Slashdot Media, is your hub for
>> all
>> things parallel software development, from weekly thought leadership blogs to
>> news, videos, case studies, tutorials and more. Take a look and join the
>> conversation now. http://goparallel.sourceforge.net/
>> <http://goparallel.sourceforge.net/>
>>
>> _______________________________________________
>> pdl-devel mailing list
>> pdl-devel@lists.sourceforge.net <mailto:pdl-devel@lists.sourceforge.net>
>> https://lists.sourceforge.net/lists/listinfo/pdl-devel
>> <https://lists.sourceforge.net/lists/listinfo/pdl-devel>
>
> ------------------------------------------------------------------------------
> Dive into the World of Parallel Programming The Go Parallel Website, sponsored
> by Intel and developed in partnership with Slashdot Media, is your hub for all
> things parallel software development, from weekly thought leadership blogs to
> news, videos, case studies, tutorials and more. Take a look and join the
> conversation now.
> http://goparallel.sourceforge.net/_______________________________________________
> pdl-devel mailing list
> pdl-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/pdl-devel
------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
pdl-devel mailing list
pdl-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/pdl-devel
