#12544: Switch cones to `PointCollection`
--------------------------------------------------+-------------------------
       Reporter:  novoselt                        |         Owner:  mhampton    
           Type:  enhancement                     |        Status:  needs_review
       Priority:  major                           |     Milestone:  sage-5.0    
      Component:  geometry                        |    Resolution:              
       Keywords:  toric                           |   Work issues:              
Report Upstream:  N/A                             |     Reviewers:              
        Authors:  Andrey Novoseltsev              |     Merged in:              
   Dependencies:  #11599, #11634, #12541, #12361  |      Stopgaps:              
--------------------------------------------------+-------------------------

Comment (by novoselt):

 After some work on switching lattice polytopes also to point collections,
 I've finally realized that it is not a good idea to have
 `cone.rays().matrix()` return matrix with rays as columns, one of the main
 reasons is that `matrix(list(cone.rays()))` gives the opposite
 representation with rays as rows. The main motivation for the original
 format was convenience to look at rays/points, but with a specialized
 container we can easily transpose the matrix in repr/latex, so I have done
 precisely that: now `matrix(cone.rays()) == cone.rays().matrix()` and
 `cone.rays()[i] == cone.rays().matrix()[i]`.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12544#comment:15>
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