On Thu, Apr 9, 2009 at 6:32 PM, mabshoff <mabsh...@googlemail.com> wrote:
>
>
>
> On Apr 9, 5:53 pm, William Stein <wst...@gmail.com> wrote:
>> On Thu, Apr 9, 2009 at 11:13 AM, davidloeffler <dave.loeff...@gmail.com> 
>> wrote:
>
> <SNIP>
>
>> Great.  I've refereed this (positively).  You fixed *numerous* bugs in
>> the code, imho, when writing those 28 new doctests -- great work!
>> Let's keep the doctest patches coming.
>
> Yep, note that this fix for David's patch is a common issue:
>
> 202                         sage: cmp(G, 1)
> 203                         -1
>        202                 sage: cmp(G, 1) in [1,-1]
>        203                 True
>
> cmp($FOO,$BAR) returns -1,0 or 1 depending on the memory address, so
> the version that William did is much cleaner since G != 1 in this
> case. A lot of those cmp() tests were set to random, but it is bad
> style to do that.
>
>> I'm mostly available on Saturday for a doctest day... so mark your calendars:
>>
>> SATURDAY, APRIL 11 -- Doctest Day.
>
> +1 - someone might want to make an official wiki page for DocDay X and
> send out an official announcement.

Thanks!  (It sounds to me like you just volunteered yourself.)

William

--~--~---------~--~----~------------~-------~--~----~
To post to this group, send email to sage-devel@googlegroups.com
To unsubscribe from this group, send email to 
sage-devel-unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/sage-devel
URLs: http://www.sagemath.org
-~----------~----~----~----~------~----~------~--~---

Reply via email to