Bill Allombert on Fri, 19 Dec 2008 23:19:57 +0100

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

Re: Functions elllderiv and ellanalyticrank

On Fri, Dec 19, 2008 at 07:20:53PM +0100, Karim Belabas wrote:
> > 2) I think elllderiv should be renamed ellL1 for consistency with bnrL1
> > (and maybe add a flag so that both functions have the same semantic).
> > 
> > 7) Please document what is VSX in #define VSX 5.
> > (Or better remove VSX. One day someone will create a CPU called VSX and
> > gcc with define VSX. It already appened with k6 and sgi.)
> > 
> Hi Bill, 
>   thanks for looking at the patch and the detailed comments. I did look
> at it too, but was too lazy to explain every single change; hence the
> 6 "cleanup" commits yesterday and this morning [ 11462 to 11468 ]
> The current svn version already incorporates all the above (and many more),
> except the second error message in 6). ( Which I believe is an
> impossible situation but had no time to check yet.)

It seems points 2) and 7) above were missed.

Also the DEBUG message should be improved to be more useful.

> There are only two items left from my initial TODO list about this
> patch:
> A) It still uses far too much memory, so that the "small" curves with
> rank 7 (say) can't be handled with a PARI stack of 1GByte.
> B) Once A) is no longer a problem (say, in about 40 minutes), check Tom
> Womack's table at
> Many interesting curves with "largish but tolerable" conductor...

I just checked the first curves for each rank <=7, and it worked fine.
For the rank 7, and curve [0,1,0,-5945,583879], we get:

%3 = [7, 10410274.011880989226208667596394924308]
  ***   last result computed in 32mn, 22,973 ms.