#18673: Replace GP by PARI in basis_of_short_vectors()
-------------------------------------+-------------------------------------
       Reporter:  jdemeyer           |        Owner:
           Type:  enhancement        |       Status:  needs_work
       Priority:  major              |    Milestone:  sage-6.9
      Component:  quadratic forms    |   Resolution:
       Keywords:                     |    Merged in:
        Authors:  Jeroen Demeyer     |    Reviewers:  Vincent Delecroix
Report Upstream:  N/A                |  Work issues:
         Branch:                     |       Commit:
  u/jdemeyer/replace_gp_by_pari_in_basis_of_short_vectors__|  
cbc929f84f4636f76f1bf6b551ef037f23c475d5
   Dependencies:  #6326              |     Stopgaps:
-------------------------------------+-------------------------------------
Changes (by vdelecroix):

 * status:  needs_review => needs_work
 * reviewer:   => Vincent Delecroix
 * milestone:  sage-6.8 => sage-6.9


Comment:

 You added a cached method to a method with a `safe_flag` argument that is
 intended to make the function returns a '''new''' copy of the object each
 time! The `cached_method` is incompatible with this `sage_flag`. One
 option is to get rid `sage_flag` (deprecation needed). And to be safe, I
 would return a '''tuple''' of immutable vectors instead of a list of
 mutable ones.

--
Ticket URL: <http://trac.sagemath.org/ticket/18673#comment:4>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to