Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the PDL-Opt-QP CPAN distribution.

Report information
The Basics
Id: 88871
Status: resolved
Priority: 0/
Queue: PDL-Opt-QP
People
Owner: Nobody in particular
Requestors: chm [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.25
Fixed in: 0.26_10

History Show all quoted text
  Sat Sep 21 14:06:22 2013 chm [...] cpan.org - Ticket created
Subject: does not support coming 64bit indexing
The SIZE() of PDL indexes, dimensions, and offsets can be
64bit if your perl platform supports it with the coming PDL-2.007
release.  A look at your PP code shows that you have assumed
that index values are all 32bits.  You'll need to change the data
type of index quantities to PDL_Indx (in C), indx (in PP signatures),
and indx type at the perl/PDL level to enable 64bit support.

Check defined(&PDL::indx) to determine if the PDL you are building
against has the new 64bit support.
  Wed Sep 25 12:52:56 2013 mgrimes [...] cpan.org - Correspondence added
Thanks for the bug report. When 2.006_07 was release, I confirmed that PDL::Opt::QP continues to pass tests, but I don't fully understand what is involved in making it support 64bit index values. I appreciate the explanation in your bug report, but can you point me to more detailed documentation? I'm not sure 64bit support is really needed for this module. This algorithm is just way too slow to handle problems with more than 2^32 inputs. Further, I doubt the underlying Fortran code supports inputs that large--and I don't have the Fortran expertise to try to fix it. On Sat Sep 21 14:06:22 2013, CHM wrote: Show quoted text
> The SIZE() of PDL indexes, dimensions, and offsets can be > 64bit if your perl platform supports it with the coming PDL-2.007 > release. A look at your PP code shows that you have assumed > that index values are all 32bits. You'll need to change the data > type of index quantities to PDL_Indx (in C), indx (in PP signatures), > and indx type at the perl/PDL level to enable 64bit support. > > Check defined(&PDL::indx) to determine if the PDL you are building > against has the new 64bit support.
  Wed Sep 25 12:52:57 2013 The RT System itself - Status changed from 'new' to 'open'
  Wed Sep 25 13:59:40 2013 devel.chm.01 [...] gmail.com - Correspondence added
Subject: Re: [rt.cpan.org #88871] does not support coming 64bit indexing
Date: Wed, 25 Sep 2013 13:59:25 -0400
To: bug-PDL-Opt-QP [...] rt.cpan.org
From: Chris Marshall <devel.chm.01 [...] gmail.com>
If it works ok, I suggest just documenting that the current implementation does not support 64bit sized piddles. You could check for defined(&PDL::indx) and issue a warning at configure/install time. Runtime warnings could also be done with the same check. Thanks for verifying that it works, Chris On Wed, Sep 25, 2013 at 12:52 PM, Mark Grimes via RT <bug-PDL-Opt-QP@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=88871 > > > Thanks for the bug report. > > When 2.006_07 was release, I confirmed that PDL::Opt::QP continues to pass tests, but I don't fully understand what is involved in making it support 64bit index values. I appreciate the explanation in your bug report, but can you point me to more detailed documentation? > > I'm not sure 64bit support is really needed for this module. This algorithm is just way too slow to handle problems with more than 2^32 inputs. Further, I doubt the underlying Fortran code supports inputs that large--and I don't have the Fortran expertise to try to fix it. > > > On Sat Sep 21 14:06:22 2013, CHM wrote:
>> The SIZE() of PDL indexes, dimensions, and offsets can be >> 64bit if your perl platform supports it with the coming PDL-2.007 >> release. A look at your PP code shows that you have assumed >> that index values are all 32bits. You'll need to change the data >> type of index quantities to PDL_Indx (in C), indx (in PP signatures), >> and indx type at the perl/PDL level to enable 64bit support. >> >> Check defined(&PDL::indx) to determine if the PDL you are building >> against has the new 64bit support.
> > > >
  Sun Dec 08 12:21:55 2013 mgrimes [...] cpan.org - Correspondence added
Documented behavior and emits a warning under 64-bit PDL.
  Sun Dec 08 12:21:56 2013 mgrimes [...] cpan.org - Status changed from 'open' to 'resolved'
  Sun Dec 08 12:21:56 2013 mgrimes [...] cpan.org - Fixed in 0.26_10 added