Changes between Version 1 and Version 2 of Ticket #11735


Ignore:
Timestamp:
09/10/11 16:17:00 (8 years ago)
Author:
ncohen
Comment:

As this patch is already written (and readily updated to be applied on top of #11738), I think it best to merge it into Sage as soon as possible. It (apparently) fixes the bug reported by Jan, but most importantly ensures that the answer returned is indeed *valid*, so that no erroneous answer could be returned as previously.

Of course, this version of the implementation will have to be patched as soon as possible using Jan's code, in order to match a proven algorithm (or until I can come up with a proof that the current implementation is correct, or find a paper that does if I actually found it somewhere).

I'll do that *asap*, but I still think it better to merge this patch in the meantime, if only to be sure no bad answer is ever returned (God I hate that).

Nathann

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #11735

    • Property Cc ddestrada added
  • Ticket #11735 – Description

    v1 v2  
    66Nathann
    77
     8Requires :
     9   * #11738
     10Apply:
     11   * [attachment:trac_11735.patch]
     12
    813[1] https://groups.google.com/d/topic/sage-support/rU1VTz1Ou_I/discussion