Hi Jason!

On Tue, Jan 12, 2010 at 10:47:01PM +0100, Nicolas M. Thiery wrote:
> > I now have a patch up at
> > 
> > http://trac.sagemath.org/sage_trac/ticket/7914

A couple details; could you:

 - rename the patch in the queue as on trac
 - Add a description to the top of the patch (for example using
   hg qrefresh -e), like:

   #7914: Implementation of triangular morphisms for modules with basis

 - extract two functions:

   def max(iterable, cmp = ...)
   def min(iterable, cmp = ...)

   possibly in sage.misc.misc_c, or maybe that should be in the new
   sage.misc.sage_itertools I introduced in #7921 (but then it will
   introduce a dependency upon it).

Thanks!

Cheers,
                                Nicolas
--
Nicolas M. ThiƩry "Isil" <nthi...@users.sf.net>
http://Nicolas.Thiery.name/
-- 
You received this message because you are subscribed to the Google Groups 
"sage-combinat-devel" group.
To post to this group, send email to sage-combinat-de...@googlegroups.com.
To unsubscribe from this group, send email to 
sage-combinat-devel+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/sage-combinat-devel?hl=en.


Reply via email to