great work, Jonathan!

I've tested, and I've found the following problems:

1) part 1  hangs

2) compiling with -Wall gives this warning

part.cc:865: warning: unused variable 'temp2'

3)  part without arguments returns 42

Pablo


On 7/29/07, Jonathan Bober <[EMAIL PROTECTED]> wrote:
> Attached is a somewhat improved version.
>
> (I don't want to flood the mailing list with attachments, so I won't
> send this file as an attachment to the list again after this.)
>
> The main improvement is that the code is better commented and more
> readable now. The secondary improvement is an increase in speed - I
> think it's about 15% faster, or something like that, at computing
> p(10^9) than the previous code. (Although I'm not exactly sure what the
> phrase "15% faster" means, and I was having some weird issues with my
> laptop when I timed the old code, so I'm not sure exactly how fast it
> ran.) There are no improvements in automated testing, so I'm no more
> confident of this correctness of this code than I was of the old code,
> but it certainly seems to work.
>
> I'm confident that this can be sped up more, but I'm not sure that we
> can reach as low as 10 seconds to compute p(10^9). It certainly might be
> possible, though.
>
> >
>
>

--~--~---------~--~----~------------~-------~--~----~
To post to this group, send email to sage-devel@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at http://groups.google.com/group/sage-devel
URLs: http://sage.scipy.org/sage/ and http://modular.math.washington.edu/sage/
-~----------~----~----~----~------~----~------~--~---

Reply via email to