#6922: Matrix term ordering
--------------------------------+-------------------------------------------
   Reporter:  klee              |       Owner:  Somebody    
       Type:  enhancement       |      Status:  needs_review
   Priority:  minor             |   Milestone:  sage-4.5    
  Component:  basic arithmetic  |    Keywords:  term order  
     Author:  Kwankyu Lee       |    Upstream:  N/A         
   Reviewer:                    |      Merged:              
Work_issues:                    |  
--------------------------------+-------------------------------------------

Comment(by malb):

 The patch applies cleanly and doctests pass.

 I'm still not happy with the interface:

 {{{
 #!python
 sage: P.<a,b> =
 PolynomialRing(GF(32003),order=TermOrder(Matrix([1,2,0,3])))
 sage: P.term_order()
 m(1,2,0,3) term order
 }}}

 This uses the non-standard "m(...)" representation which I would avoid.
 I'd be happy with either "M()" (Singular notation) or "Matrix term
 ordering with matrix ..." or so.

 Also, the "m()" notation is allowed but shouldn't.

 {{{
 #!python
 sage: P.<a,b> = PolynomialRing(GF(32003),order='m(1,2,0,3)')
 }}}

 I understand that this is a typical paint-the-bike-shed scenario and in
 particular a question of choice. Still, I think we shouldn't invent more
 ad-hoc string notation when (a) there is an established notation and (b)
 we have a much nicer object oriented way of constructing term orderings.

 However, since this isn't really that big of a deal, I am okay with being
 overruled by some other referee who disagrees.

 PS: Apologies for taking so long to revisit this ticket.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6922#comment:17>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.

Reply via email to